Skip to content

Add GitHub Actions workflow for SecretShield - #2

Merged
samarthchugh0602 merged 1 commit into
mainfrom
Sam3360-Add-SS
Sep 18, 2026
Merged

samarthchugh0602 merged 1 commit into
mainfrom
Sam3360-Add-SS

Conversation

@Sam3360

@Sam3360 Sam3360 commented Sep 17, 2026

Copy link
Copy Markdown
Member

This workflow automates the installation and scanning of the SecretShield tool on push and pull request events.

Before approving kindly configure the .secretshieldignore file

THANKS :)

This workflow automates the installation and scanning of the SecretShield tool on push and pull request events.
@Sam3360 Sam3360 added the documentation Improvements or additions to documentation label Sep 17, 2026

@coolguy173 coolguy173 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice work samarth(or claude)

@Sam3360

Sam3360 commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

@coolguy173 ts not claude buddy 😭

@Sam3360

Sam3360 commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

also i didnt put anything in the .secretshieldignore yestersay cuz i didnt have time gng

@samarthchugh0602
samarthchugh0602 merged commit 62c6ad7 into main Sep 18, 2026
0 of 3 checks passed
@samarthchugh0602

Copy link
Copy Markdown
Member

YESSSS doneeee

@samarthchugh0602 samarthchugh0602 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅

@Daniel234775

Copy link
Copy Markdown
Member

just tested the edit, it fixed most of the stuff, the only thing that failed was test_real_git_commit_succeeds_when_clean which is the same windows path issue i found last time

@Sam3360

Sam3360 commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

which version u on?
i released 0.5.2 which added .secretshieldignore (works just like gitignore but like this one tells ss what to not check) and like colouredd output for static scanning (like cmd or powershell)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants