Skip to content

Socket Basics SAST: add workflow and gitignore entries#29

Merged
buffym merged 3 commits into
mainfrom
stagility/socket-basics-sast
Jul 2, 2026
Merged

Socket Basics SAST: add workflow and gitignore entries#29
buffym merged 3 commits into
mainfrom
stagility/socket-basics-sast

Conversation

@sgrammargs

@sgrammargs sgrammargs commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds .github/workflows/socket-basics.yml — calls the shared Socket Basics reusable workflow at @main
  • Adds .socket-scans/ and .socket.facts.json to .gitignore

Dependencies

Requires ynab/shared-actions#233 to be merged first.

Test plan

  • Trigger on a PR — verify the socket-basics-security-scan job runs and posts a comment
  • Verify no unexpected findings block the PR

🤖 Generated with Claude Code

sgrammargs and others added 2 commits June 25, 2026 16:41
@sgrammargs sgrammargs marked this pull request as ready for review June 30, 2026 13:44
@sgrammargs sgrammargs requested a review from buffym June 30, 2026 19:24
The shared workflow (ynab/shared-actions#233) is gaining a required
GH_YBOT_TOKEN secret (it now checks out shared-actions to run its helper
actions from current main). `secrets: inherit` forwards the org secrets it
needs today and means this file never needs touching when the shared
workflow's secret needs change again.

Safe to merge immediately: inherit also satisfies the current version of
the shared workflow.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@buffym buffym merged commit 462dc56 into main Jul 2, 2026
7 checks passed
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.

2 participants