Skip to content

Unpushed commits (local main ahead of origin/main)#211

Merged
StanBarrows merged 28 commits intomainfrom
feature-dependency-bump
Apr 7, 2026
Merged

Unpushed commits (local main ahead of origin/main)#211
StanBarrows merged 28 commits intomainfrom
feature-dependency-bump

Conversation

@StanBarrows
Copy link
Copy Markdown
Contributor

Local main was 28 commit(s) ahead of origin/main. Opened from update-opensource-active.sh for review.

Copilot AI review requested due to automatic review settings March 26, 2026 07:15
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a GitHub Actions workflow to run Composer security auditing when PHP dependencies change and on a scheduled basis, improving automated supply-chain security checks for the PHP/Laravel parts of the repo.

Changes:

  • Introduces a new Composer audit workflow triggered on composer.json/composer.lock changes, weekly schedule, and manual dispatch.
  • Sets up PHP and runs Composer commands to resolve dependencies and perform composer audit.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +36 to +37
composer update --no-interaction --no-ansi
composer audit
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

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

composer update will modify the dependency set during CI and can make this audit check different packages than the ones actually locked/used by the repo. For a security audit workflow, this should install from composer.lock (e.g., composer install or ramsey/composer-install) and run composer audit --locked so findings reflect the committed lockfile.

Suggested change
composer update --no-interaction --no-ansi
composer audit
composer install --no-interaction --no-ansi
composer audit --locked

Copilot uses AI. Check for mistakes.
@StanBarrows StanBarrows merged commit 979c3c6 into main Apr 7, 2026
16 checks passed
@StanBarrows StanBarrows deleted the feature-dependency-bump branch April 7, 2026 14:02
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