Skip to content

Comments

Sec25 85 add psalm scanning#7

Open
anthonymenefee wants to merge 3 commits intomasterfrom
SEC25-85-add-psalm-scanning
Open

Sec25 85 add psalm scanning#7
anthonymenefee wants to merge 3 commits intomasterfrom
SEC25-85-add-psalm-scanning

Conversation

@anthonymenefee
Copy link

This PR adds psalm workflows to this repository for ongoing security and quality scanning.

Adding psalm to this repository.
Change to 4
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@anthonymenefee
Copy link
Author

@b-hayes Tagging you in this series of PRs for psalm. Sorry for the noise. Let us know if you think this is good to go as is or if the repo needs some special attention. Thanks!

@b-hayes
Copy link
Contributor

b-hayes commented Feb 10, 2026

@anthonymenefee and @joshmcrae this repo will probably never be updated again. It's only used for the legacy deployment method and will likely be discarded once we move to ECS.

@b-hayes
Copy link
Contributor

b-hayes commented Feb 10, 2026

PS - It technically doesn't matter to me if you merge it either since it won't affect anything.

@b-hayes
Copy link
Contributor

b-hayes commented Feb 10, 2026

I noticed there is no baseline file. Should be running vendor/bin/psalm --set-baseline for these repos unless there was an executive decision not to do so.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants