Skip to content

chore: adding disable ignore flag#183

Open
Bill Li (billxinli) wants to merge 1 commit intomainfrom
chore/disable-ignore
Open

chore: adding disable ignore flag#183
Bill Li (billxinli) wants to merge 1 commit intomainfrom
chore/disable-ignore

Conversation

@billxinli
Copy link
Copy Markdown

@billxinli Bill Li (billxinli) commented Apr 9, 2026

  • Adds --disable-ignore CLI flag that disables support for Socket (@SocketSecurity) ignore commands in PR comments
  • When set, alerts cannot be suppressed via PR comments and ignore instructions are hidden from comment output

Changes

  • config.py: New disable_ignore boolean field on CliConfig, argparse flag --disable-ignore
  • socketcli.py: Skips comment-based alert suppression in both comment-initiated and push/diff flows when flag is set
  • messages.py: Conditionally hides "Mark as acceptable risk" / Socket (@SocketSecurity) ignore instructions from PR comment output
  • docs/cli-reference.md: Documents the new flag in usage synopsis and Advanced Configuration table

Public Changelog

N/A

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026 •
edited
Loading

🚀 Preview package published!

Install with:

pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple socketsecurity==2.2.81.dev8

Docker image: socketdev/cli:pr-183

@billxinli Bill Li (billxinli) marked this pull request as ready for review April 9, 2026 16:41
@billxinli Bill Li (billxinli) requested a review from a team as a code owner April 9, 2026 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@billxinli