Skip to content

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

Merged
StanBarrows merged 24 commits intomainfrom
feature-dependency-bump
Apr 9, 2026
Merged

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

Conversation

@StanBarrows
Copy link
Copy Markdown
Contributor

Local main was 24 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:17
Copy link
Copy Markdown

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 dedicated GitHub Actions workflow to run Composer security audits when dependency manifests change and on a weekly schedule.

Changes:

  • Introduces a new Composer audit workflow triggered on composer.json/composer.lock changes, schedule, and manual dispatch.
  • Sets up PHP 8.4 and runs Composer commands to resolve dependencies and 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.

Running composer update before composer audit means the audit is performed against freshly-updated dependencies rather than the versions in composer.lock. This can let a vulnerable version that’s committed in the lockfile pass CI because the workflow upgrades it in the runner. Prefer auditing the lockfile/installed locked deps (e.g., install from composer.lock and audit with the locked set) so the check reflects what’s actually in the repo.

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

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

3 participants