| route-pattern | /_view_fragments/voltron/pull_requests/show/:user_id/:repository/:id/pull_request_layout(.:format) |
| route-controller | voltron_pull_requests_fragments |
| route-action | pull_request_layout |
| fetch-nonce | v2:d9043cd0-a6f6-c11a-ec87-6037e18f8842 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | A89C:FDC34:93A6648:C35FA04:696E0455 |
| html-safe-nonce | 9febee35b23096a00b42329ea5bbc09b583205face3624365f18d0e4d03dcdbf |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBODlDOkZEQzM0OjkzQTY2NDg6QzM1RkEwNDo2OTZFMDQ1NSIsInZpc2l0b3JfaWQiOiIzNzUwOTgxODM2ODY3NTAzMTg5IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | c173b74ac209358644d5184aa90fdd4ab56612b0ac8bfb266bf432a26759ee4d |
| hovercard-subject-tag | pull_request:2243253086 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | ///voltron/pull_requests_fragments/pull_request_layout |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/voltron/pull_requests/show/cpp-linter/cpp-linter/135/pull_request_layout |
| twitter:image | https://opengraph.githubassets.com/de58175ad9f313ffd76199aaf88404c6c8a2f0702800886675ecb4310b9262b4/cpp-linter/cpp-linter/pull/135 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/de58175ad9f313ffd76199aaf88404c6c8a2f0702800886675ecb4310b9262b4/cpp-linter/cpp-linter/pull/135 |
| og:image:alt | This is an upstream continuation from #131 with a focus on rewriting the PR review tests. |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | 2bndy5 |
| hostname | github.com |
| expected-hostname | github.com |
| None | 9b5131b207ddd175abf059a848d5f4302ec0606b02211b989013be49cf08593e |
| turbo-cache-control | no-preview |
| go-import | github.com/cpp-linter/cpp-linter git https://github.com/cpp-linter/cpp-linter.git |
| octolytics-dimension-user_id | 103884627 |
| octolytics-dimension-user_login | cpp-linter |
| octolytics-dimension-repository_id | 523671868 |
| octolytics-dimension-repository_nwo | cpp-linter/cpp-linter |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 523671868 |
| octolytics-dimension-repository_network_root_nwo | cpp-linter/cpp-linter |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | f8590a63bfc8093b241930ca57d536c9a50f9680 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
| Skip to content | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#start-of-content |
|
| https://patch-diff.githubusercontent.com/ |
|
Sign in
| https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fcpp-linter%2Fcpp-linter%2Fpull%2F135 |
| GitHub CopilotWrite better code with AI | https://github.com/features/copilot |
| GitHub SparkBuild and deploy intelligent apps | https://github.com/features/spark |
| GitHub ModelsManage and compare prompts | https://github.com/features/models |
| MCP RegistryNewIntegrate external tools | https://github.com/mcp |
| ActionsAutomate any workflow | https://github.com/features/actions |
| CodespacesInstant dev environments | https://github.com/features/codespaces |
| IssuesPlan and track work | https://github.com/features/issues |
| Code ReviewManage code changes | https://github.com/features/code-review |
| GitHub Advanced SecurityFind and fix vulnerabilities | https://github.com/security/advanced-security |
| Code securitySecure your code as you build | https://github.com/security/advanced-security/code-security |
| Secret protectionStop leaks before they start | https://github.com/security/advanced-security/secret-protection |
| Why GitHub | https://github.com/why-github |
| Documentation | https://docs.github.com |
| Blog | https://github.blog |
| Changelog | https://github.blog/changelog |
| Marketplace | https://github.com/marketplace |
| View all features | https://github.com/features |
| Enterprises | https://github.com/enterprise |
| Small and medium teams | https://github.com/team |
| Startups | https://github.com/enterprise/startups |
| Nonprofits | https://github.com/solutions/industry/nonprofits |
| App Modernization | https://github.com/solutions/use-case/app-modernization |
| DevSecOps | https://github.com/solutions/use-case/devsecops |
| DevOps | https://github.com/solutions/use-case/devops |
| CI/CD | https://github.com/solutions/use-case/ci-cd |
| View all use cases | https://github.com/solutions/use-case |
| Healthcare | https://github.com/solutions/industry/healthcare |
| Financial services | https://github.com/solutions/industry/financial-services |
| Manufacturing | https://github.com/solutions/industry/manufacturing |
| Government | https://github.com/solutions/industry/government |
| View all industries | https://github.com/solutions/industry |
| View all solutions | https://github.com/solutions |
| AI | https://github.com/resources/articles?topic=ai |
| Software Development | https://github.com/resources/articles?topic=software-development |
| DevOps | https://github.com/resources/articles?topic=devops |
| Security | https://github.com/resources/articles?topic=security |
| View all topics | https://github.com/resources/articles |
| Customer stories | https://github.com/customer-stories |
| Events & webinars | https://github.com/resources/events |
| Ebooks & reports | https://github.com/resources/whitepapers |
| Business insights | https://github.com/solutions/executive-insights |
| GitHub Skills | https://skills.github.com |
| Documentation | https://docs.github.com |
| Customer support | https://support.github.com |
| Community forum | https://github.com/orgs/community/discussions |
| Trust center | https://github.com/trust-center |
| Partners | https://github.com/partners |
| GitHub SponsorsFund open source developers | https://github.com/sponsors |
| Security Lab | https://securitylab.github.com |
| Maintainer Community | https://maintainers.github.com |
| Accelerator | https://github.com/accelerator |
| Archive Program | https://archiveprogram.github.com |
| Topics | https://github.com/topics |
| Trending | https://github.com/trending |
| Collections | https://github.com/collections |
| Enterprise platformAI-powered developer platform | https://github.com/enterprise |
| GitHub Advanced SecurityEnterprise-grade security features | https://github.com/security/advanced-security |
| Copilot for BusinessEnterprise-grade AI features | https://github.com/features/copilot/copilot-business |
| Premium SupportEnterprise-grade 24/7 support | https://github.com/premium-support |
| Pricing | https://github.com/pricing |
| Search syntax tips | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
| documentation | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
|
Sign in
| https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fcpp-linter%2Fcpp-linter%2Fpull%2F135 |
|
Sign up
| https://patch-diff.githubusercontent.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fpull_requests_fragments%2Fpull_request_layout&source=header-repo&source_repo=cpp-linter%2Fcpp-linter |
| Reload | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
| Reload | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
| Reload | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
|
cpp-linter
| https://patch-diff.githubusercontent.com/cpp-linter |
| cpp-linter | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter |
| Please reload this page | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
|
Notifications
| https://patch-diff.githubusercontent.com/login?return_to=%2Fcpp-linter%2Fcpp-linter |
|
Fork
10
| https://patch-diff.githubusercontent.com/login?return_to=%2Fcpp-linter%2Fcpp-linter |
|
Star
34
| https://patch-diff.githubusercontent.com/login?return_to=%2Fcpp-linter%2Fcpp-linter |
|
Code
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter |
|
Issues
6
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/issues |
|
Pull requests
2
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pulls |
|
Discussions
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/discussions |
|
Actions
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/actions |
|
Projects
0
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/projects |
|
Security
Uh oh!
There was an error while loading. Please reload this page.
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/security |
| Please reload this page | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
|
Insights
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pulse |
|
Code
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter |
|
Issues
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/issues |
|
Pull requests
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pulls |
|
Discussions
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/discussions |
|
Actions
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/actions |
|
Projects
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/projects |
|
Security
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/security |
|
Insights
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pulse |
| Sign up for GitHub
| https://patch-diff.githubusercontent.com/signup?return_to=%2Fcpp-linter%2Fcpp-linter%2Fissues%2Fnew%2Fchoose |
| terms of service | https://docs.github.com/terms |
| privacy statement | https://docs.github.com/privacy |
| Sign in | https://patch-diff.githubusercontent.com/login?return_to=%2Fcpp-linter%2Fcpp-linter%2Fissues%2Fnew%2Fchoose |
| Jump to bottom | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#issue-comment-box |
| 2bndy5 | https://patch-diff.githubusercontent.com/2bndy5 |
| main | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/tree/main |
| hide-reuse-pr-reviews | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/tree/hide-reuse-pr-reviews |
|
Hide or reuse PR reviews
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#top |
| 2bndy5 | https://patch-diff.githubusercontent.com/2bndy5 |
| main | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/tree/main |
| hide-reuse-pr-reviews | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/tree/hide-reuse-pr-reviews |
|
Conversation
1
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
|
Commits
37
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits |
|
Checks
132
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/checks |
|
Files changed
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/files |
| Please reload this page | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
| https://github.co/hiddenchars |
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/{{ revealButtonHref }} |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| 2bndy5 | https://patch-diff.githubusercontent.com/2bndy5 |
| Dec 18, 2024 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#issue-2748921074 |
| Please reload this page | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
| #131 | https://github.com/cpp-linter/cpp-linter/pull/131 |
| Please reload this page | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
| maoliver-amd | https://patch-diff.githubusercontent.com/maoliver-amd |
| December 18, 2024 12:56 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#commits-pushed-e9d7252 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Add reuse/deletion of existing review comments | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/e9d7252f2ffd5859838b451afdbdcf2f62eb59d6 |
| e9d7252 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/e9d7252f2ffd5859838b451afdbdcf2f62eb59d6 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Add options for controlling review comment reuse/deletion. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/c96c375cb868509d544c578beefdcc16dd36905e |
| c96c375 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/c96c375cb868509d544c578beefdcc16dd36905e |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Update review comment CLI options. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/9f154070506c4ed3f52ce1a91fc8a0378ee5a8c3 |
| 9f15407 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/9f154070506c4ed3f52ce1a91fc8a0378ee5a8c3 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Remove unused imports. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/63bad773db298ead336119188a7eb8946941d3df |
| 63bad77 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/63bad773db298ead336119188a7eb8946941d3df |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Apply suggestions from code review | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/dd8eb389548da4fdf6cb6a0817bbaaf6262cd20e |
| dd8eb38 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/dd8eb389548da4fdf6cb6a0817bbaaf6262cd20e |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Use get instead of inline conditionals. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/c6e2d19674ea7d48c8c7f1a851478eef5acb48b7 |
| c6e2d19 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/c6e2d19674ea7d48c8c7f1a851478eef5acb48b7 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Reformat code. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/d991263d0cfd087fa7e5e570a174568943919467 |
| d991263 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/d991263d0cfd087fa7e5e570a174568943919467 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Make graphQL queries constants. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/263fa86150ba6c887494d8c33ebce26109a6e996 |
| 263fa86 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/263fa86150ba6c887494d8c33ebce26109a6e996 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Use self.api for graphql | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/8d63b6884a929d3f58e7f17cd9d6557c71d66670 |
| 8d63b68 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/8d63b6884a929d3f58e7f17cd9d6557c71d66670 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Set delete_comment default to False. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/a7b0f348490ac7308baaad55c03fd7cd8339a822 |
| a7b0f34 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/a7b0f348490ac7308baaad55c03fd7cd8339a822 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Rename variables to match CLI option names. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/8239d32c7e4069565b23c417da6a826539d11ed7 |
| 8239d32 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/8239d32c7e4069565b23c417da6a826539d11ed7 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Simplify retrieving line_start/line_end from graphql response. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/c6348f502ad9c5eabd30935a3f3793c678fcc0bb |
| c6348f5 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/c6348f502ad9c5eabd30935a3f3793c678fcc0bb |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Fix line_start and line_end | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/2b73f2c9f1a374308d2048e649e6340a2e6a9e27 |
| 2b73f2c | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/2b73f2c9f1a374308d2048e649e6340a2e6a9e27 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Improve review thread comment log output. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/62f62fea9d22331c14eae4f3e99c2edb9870bf4c |
| 62f62fe | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/62f62fea9d22331c14eae4f3e99c2edb9870bf4c |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Add graphql mock responses. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/687772ef52f35c54722315f22154e3988e6c43af |
| 687772e | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/687772ef52f35c54722315f22154e3988e6c43af |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Fix id tag naming. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/b1122f68fd2bd8d5f8e2efb51e60651f18abacff |
| b1122f6 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/b1122f68fd2bd8d5f8e2efb51e60651f18abacff |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Apply suggestions from code review | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/e7f13f73dc9fca24fe278a33dab2c44739481018 |
| e7f13f7 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/e7f13f73dc9fca24fe278a33dab2c44739481018 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/marketplace/coderabbitai |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Apply suggestions from code review | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/20e3d65e9ddb985c75d889a4b87ea580d103c2f7 |
| 20e3d65 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/20e3d65e9ddb985c75d889a4b87ea580d103c2f7 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Formatting cleanups. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/e9abe5c6e8860b41148112e86567a4ad2e2a7168 |
| e9abe5c | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/e9abe5c6e8860b41148112e86567a4ad2e2a7168 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Improve qraphQL query error detection. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/e978e70ae9a7aaa364011da2c26422b3076fd862 |
| e978e70 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/e978e70ae9a7aaa364011da2c26422b3076fd862 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Remove unused graphql author field | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/b0a9bd614e7b0f553138a02f20a44b11a609b075 |
| b0a9bd6 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/b0a9bd614e7b0f553138a02f20a44b11a609b075 |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| code review | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/dfb80f34578787ff4e6b6e6718136a09240932be |
| dfb80f3 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/dfb80f34578787ff4e6b6e6718136a09240932be |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Dont dismiss ignored reviews. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/70294edca50df2c47d18d04cba18e51523e04bea |
| 70294ed | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/70294edca50df2c47d18d04cba18e51523e04bea |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Correct output concern count | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/20a541e52a6a333b2602e816c231bed5cc469837 |
| 20a541e | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/20a541e52a6a333b2602e816c231bed5cc469837 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Add default argument to _dismiss_stale_reviews. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/ea6c5d1a47e0aa2ee5143a71e49b08730819cc9f |
| ea6c5d1 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/ea6c5d1a47e0aa2ee5143a71e49b08730819cc9f |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Extract _check_reused_comments into standalone function. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/3ca65bf0f99e3059c7de01cea6868cdce29f8ba8 |
| 3ca65bf | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/3ca65bf0f99e3059c7de01cea6868cdce29f8ba8 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Fix incorrect comment check. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/b244350ed1ffa1a293b142d8c044a3adbc47ca08 |
| b244350 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/b244350ed1ffa1a293b142d8c044a3adbc47ca08 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Update _close_review_comment log. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/d0a9e477eaece301b06434a203f02473521ecac2 |
| d0a9e47 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/d0a9e477eaece301b06434a203f02473521ecac2 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Fix _dismiss_stale_reviews default argument to None. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/83ff91e67ebc88f0ecbc3800680794824bf75ca6 |
| 83ff91e | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/83ff91e67ebc88f0ecbc3800680794824bf75ca6 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Apply suggestions from code review | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/c56813c2dfe6251709a469da4d959f26ecc0b42a |
| c56813c | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/c56813c2dfe6251709a469da4d959f26ecc0b42a |
| maoliver-amd | https://patch-diff.githubusercontent.com/maoliver-amd |
| December 18, 2024 12:56 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#commits-pushed-dc3ea16 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Add graphql return check. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/dc3ea164e3e25ade17320e0a5c84e5e6feeff2e0 |
| dc3ea16 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/dc3ea164e3e25ade17320e0a5c84e5e6feeff2e0 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Remove CLI option for reuse_review_comments. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/988eb75dbb00a2a73d98fe3c77f7f67816545cfb |
| 988eb75 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/988eb75dbb00a2a73d98fe3c77f7f67816545cfb |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Remove reuse_review_comments functionality. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/a577087d0f5cd57d2bfb7a4a89e9ee36ad03242f |
| a577087 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/a577087d0f5cd57d2bfb7a4a89e9ee36ad03242f |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| Change effects to affects. | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/81eba6942c7bf843ea3a242856935b4a9b91e3de |
| 81eba69 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/81eba6942c7bf843ea3a242856935b4a9b91e3de |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| fix test | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/aff2670ff70bf51fffd8365d9506dbbcf45f0e6c |
| aff2670 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/aff2670ff70bf51fffd8365d9506dbbcf45f0e6c |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| use paginated graphQL query to get review threads | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/e5f34824b9a91dc95c22651dab924e10f5c70e1f |
| e5f3482 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/e5f34824b9a91dc95c22651dab924e10f5c70e1f |
| https://patch-diff.githubusercontent.com/2bndy5 |
| 2bndy5 | https://patch-diff.githubusercontent.com/2bndy5 |
| Dec 18, 2024 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#event-15709934679 |
| https://patch-diff.githubusercontent.com/2bndy5 |
| 2bndy5 | https://patch-diff.githubusercontent.com/2bndy5 |
| force-pushed | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/compare/8fd3b39d2f3864121d6eaa4afac9bde366b01ee0..e5f34824b9a91dc95c22651dab924e10f5c70e1f |
| 8fd3b39 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/commit/8fd3b39d2f3864121d6eaa4afac9bde366b01ee0 |
| e5f3482 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/commit/e5f34824b9a91dc95c22651dab924e10f5c70e1f |
|
Compare
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/compare/8fd3b39d2f3864121d6eaa4afac9bde366b01ee0..e5f34824b9a91dc95c22651dab924e10f5c70e1f |
| December 18, 2024 23:28 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#event-15709936267 |
| https://patch-diff.githubusercontent.com/2bndy5 |
| 2bndy5 | https://patch-diff.githubusercontent.com/2bndy5 |
| December 18, 2024 23:31 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#event-15709953129 |
| https://patch-diff.githubusercontent.com/2bndy5 |
| 2bndy5 | https://patch-diff.githubusercontent.com/2bndy5 |
|
enhancement
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/issues?q=state%3Aopen%20label%3Aenhancement |
| Dec 18, 2024 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#event-15709954374 |
| https://patch-diff.githubusercontent.com/apps/codecov |
| codecov | https://patch-diff.githubusercontent.com/apps/codecov |
| Dec 18, 2024 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#issuecomment-2552473183 |
| Please reload this page | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
| Codecov | https://app.codecov.io/gh/cpp-linter/cpp-linter/pull/135?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter |
| (239d6b8) | https://app.codecov.io/gh/cpp-linter/cpp-linter/commit/239d6b8a8c99c00aefc84bfa4a4e735bf73ab8e3?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter |
| (822a845) | https://app.codecov.io/gh/cpp-linter/cpp-linter/commit/822a845270383eed9586401c03fe1ba013f31cc2?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter |
| Files with missing lines | https://app.codecov.io/gh/cpp-linter/cpp-linter/pull/135?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter |
| cpp_linter/rest_api/github_api.py | https://app.codecov.io/gh/cpp-linter/cpp-linter/pull/135?src=pr&el=tree&filepath=cpp_linter%2Frest_api%2Fgithub_api.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter#diff-Y3BwX2xpbnRlci9yZXN0X2FwaS9naXRodWJfYXBpLnB5 |
| 40 Missing ⚠️ | https://app.codecov.io/gh/cpp-linter/cpp-linter/pull/135?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter |
| cpp_linter/clang_tools/patcher.py | https://app.codecov.io/gh/cpp-linter/cpp-linter/pull/135?src=pr&el=tree&filepath=cpp_linter%2Fclang_tools%2Fpatcher.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter#diff-Y3BwX2xpbnRlci9jbGFuZ190b29scy9wYXRjaGVyLnB5 |
| 26 Missing ⚠️ | https://app.codecov.io/gh/cpp-linter/cpp-linter/pull/135?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter |
| tests/reviews/test_pr_review.py | https://app.codecov.io/gh/cpp-linter/cpp-linter/pull/135?src=pr&el=tree&filepath=tests%2Freviews%2Ftest_pr_review.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter#diff-dGVzdHMvcmV2aWV3cy90ZXN0X3ByX3Jldmlldy5weQ== |
| 2 Missing ⚠️ | https://app.codecov.io/gh/cpp-linter/cpp-linter/pull/135?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter |
| ☔ View full report in Codecov by Sentry | https://app.codecov.io/gh/cpp-linter/cpp-linter/pull/135?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter |
| Share it here | https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cpp-linter |
| Please reload this page | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
| https://patch-diff.githubusercontent.com/2bndy5 |
| 2bndy5 | https://patch-diff.githubusercontent.com/2bndy5 |
| force-pushed | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/compare/d411d65cd89561b67e81b96ad7b4706a663dd83a..07681019b52a9af8348a6a86d6984cc0ea1a6b7d |
| d411d65 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/commit/d411d65cd89561b67e81b96ad7b4706a663dd83a |
| 0768101 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/commit/07681019b52a9af8348a6a86d6984cc0ea1a6b7d |
|
Compare
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/compare/d411d65cd89561b67e81b96ad7b4706a663dd83a..07681019b52a9af8348a6a86d6984cc0ea1a6b7d |
| December 23, 2024 05:59 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#event-15744259463 |
| https://patch-diff.githubusercontent.com/2bndy5 |
| 2bndy5 | https://patch-diff.githubusercontent.com/2bndy5 |
|
Dec 23, 2024
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#ref-pullrequest-2755442039 |
|
update preview pictures
cpp-linter/cpp-linter-action#289
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter-action/pull/289 |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
| deeper review | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/822a845270383eed9586401c03fe1ba013f31cc2 |
| 822a845 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135/commits/822a845270383eed9586401c03fe1ba013f31cc2 |
| https://patch-diff.githubusercontent.com/2bndy5 |
| 2bndy5 | https://patch-diff.githubusercontent.com/2bndy5 |
| force-pushed | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/compare/07681019b52a9af8348a6a86d6984cc0ea1a6b7d..822a845270383eed9586401c03fe1ba013f31cc2 |
| 0768101 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/commit/07681019b52a9af8348a6a86d6984cc0ea1a6b7d |
| 822a845 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/commit/822a845270383eed9586401c03fe1ba013f31cc2 |
|
Compare
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/compare/07681019b52a9af8348a6a86d6984cc0ea1a6b7d..822a845270383eed9586401c03fe1ba013f31cc2 |
| December 23, 2024 20:51 | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135#event-15751614897 |
| Sign up for free | https://patch-diff.githubusercontent.com/join?source=comment-repo |
| Sign in to comment | https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fcpp-linter%2Fcpp-linter%2Fpull%2F135 |
|
enhancement
| https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/issues?q=state%3Aopen%20label%3Aenhancement |
| Please reload this page | https://patch-diff.githubusercontent.com/cpp-linter/cpp-linter/pull/135 |
|
| https://patch-diff.githubusercontent.com/2bndy5 |
|
| https://patch-diff.githubusercontent.com/maoliver-amd |
|
| https://patch-diff.githubusercontent.com/apps/copilot-swe-agent |
|
| https://github.com |
| Terms | https://docs.github.com/site-policy/github-terms/github-terms-of-service |
| Privacy | https://docs.github.com/site-policy/privacy-policies/github-privacy-statement |
| Security | https://github.com/security |
| Status | https://www.githubstatus.com/ |
| Community | https://github.community/ |
| Docs | https://docs.github.com/ |
| Contact | https://support.github.com?tags=dotcom-footer |