Visitar URL original
Remove the roave/security-advisories workaround by schlessera · Pull Request #309 · wp-cli/.github · GitHub
Skip to content

Remove the roave/security-advisories workaround - #309

Merged
schlessera merged 1 commit into
mainfrom
remove-roave-workaround
Oct 30, 2026
Merged

schlessera merged 1 commit into
mainfrom
remove-roave-workaround

Conversation

@schlessera

Copy link
Copy Markdown
Member

Part of #308.

#305 removes roave/security-advisories in CI when it conflicts with the running PHP version. Once the package is gone from all repositories, the step has nothing left to do, so this removes it from the unit and functional workflows. Composer's built-in advisory blocking now does the same check on every PHP version, without rejecting PHP itself.

The Windows update: true from #305 stays: testing on the latest patch release is useful on its own.

Merge this last. Until each repository has dropped the package, the step keeps their old-PHP jobs working.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@schlessera
schlessera requested a review from a team as a code owner October 30, 2026 18:35
@coderabbitai

coderabbitai Bot commented Oct 30, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 142d5b20-c328-4c84-b487-61051c88a4fc
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added enhancement New feature or request scope:testing labels Oct 30, 2026
@schlessera
schlessera merged commit 11b2b5c into main Oct 30, 2026
16 checks passed
@schlessera
schlessera deleted the remove-roave-workaround branch October 30, 2026 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request scope:testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant