Skip to content

Fix readme file - #24

Merged
akanthed merged 4 commits into
mainfrom
fix-readme-file
Aug 26, 2026
Merged

Fix readme file#24
akanthed merged 4 commits into
mainfrom
fix-readme-file

Conversation

@akanthed

Copy link
Copy Markdown
Owner

What and why

Detection logic changes (delete this section if not applicable)

  • npm run regression was run and every proven/likely finding it printed was reviewed against its source line
  • A new/updated rule has fixtures in both test-fixtures/vulnerable/ and test-fixtures/safe/
  • src/scanner/catalog.ts and src/scanner/explainer.ts both have an entry for any new rule ID
  • test/corpus.test.js's EXPECTED_VULNERABLE was updated for any new rule

CLI changes (delete this section if not applicable)

  • test/cli.test.js has a new or updated case exercising the built binary (src/cli.ts changes aren't covered by any other test file)

Checklist

  • npm run build && npm test passes
  • Before/after behavior is described above, with a concrete example if the change affects scan output

@akanthed
akanthed merged commit 5fe9187 into main Aug 26, 2026
11 checks passed
@akanthed akanthed mentioned this pull request Aug 26, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant