[Demo] C416 false positive with sequences as comprehension elements - #17
Open
tkim602 wants to merge 1 commit into
Open
[Demo] C416 false positive with sequences as comprehension elements#17tkim602 wants to merge 1 commit into
tkim602 wants to merge 1 commit into
Conversation
PatchCheck reviewModel signal: LOWER Code-change findings
Fingerprints
Review-priority signal only; not a safety approval. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #12
Demo provenance
This live replay reproduces a historical open-source issue and candidate patch, then runs it through the deployed frozen PatchCheck verifier.
1352f9be1aef608917258115d92eeb8856df5b8a938b26c57b0bad9742037501GitHub canonicalizes unchanged hunk context when rendering the pull request diff; the base file and candidate code edit are reproduced exactly.
Selection was fixed in
docs/demo-cases.jsonbefore inference. PatchCheck is advisory and does not merge or approve this patch.