Skip to content

[Demo] C416 false positive with sequences as comprehension elements - #17

Open
tkim602 wants to merge 1 commit into
demo/flake8-comprehensions-basefrom
demo/flake8-comprehensions
Open

[Demo] C416 false positive with sequences as comprehension elements#17
tkim602 wants to merge 1 commit into
demo/flake8-comprehensions-basefrom
demo/flake8-comprehensions

Conversation

@tkim602

@tkim602 tkim602 commented Aug 16, 2026

Copy link
Copy Markdown
Owner

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.

GitHub 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.json before inference. PatchCheck is advisory and does not merge or approve this patch.

@github-actions

github-actions Bot commented Aug 16, 2026

Copy link
Copy Markdown

PatchCheck review

Model signal: LOWER
Model risk percentile: 28.2

Code-change findings

  • medium: production Python changed without test-file changes (src/flake8_comprehensions.py)
Fingerprints
  • base_sha: 766ec5ea4637a3d56d9fc1dd7c9bbfbe4c61b7d7
  • head_sha: 5b40126225eb6324c202b3238727362529825637

Review-priority signal only; not a safety approval.

@tkim602 tkim602 added the demo Live replay of a frozen ChangeGuard example label Aug 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

demo Live replay of a frozen ChangeGuard example

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant