Skip to content

Update wp-cli framework#1082

Closed
schlessera wants to merge 3 commits into
mainfrom
update-framework
Closed

Update wp-cli framework#1082
schlessera wants to merge 3 commits into
mainfrom
update-framework

Conversation

@schlessera

@schlessera schlessera commented Jul 21, 2026

Copy link
Copy Markdown
Member

This is an automated pull-request

Updates the wp-cli/wp-cli framework to the latest changeset.

Summary by CodeRabbit

  • Chores
    • Improved automated pull request detection in the framework update workflow.
    • Ensured the workflow reliably chooses between creating a new pull request and updating an existing one.

@schlessera
schlessera requested a review from a team as a code owner July 21, 2026 14:31
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The workflow now uses structured gh pr list JSON output to determine whether an open framework update pull request exists.

Changes

Framework update workflow

Layer / File(s) Summary
Structured pull request number detection
.github/workflows/update-framework.yml
The workflow extracts the first open pull request number with --json number and --jq, replacing template-based parsing.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: swissspidy

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the PR’s main goal of updating the wp-cli framework dependency.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update-framework

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

@github-actions github-actions Bot added the enhancement New feature or request label Jul 21, 2026
@codecov

codecov Bot commented Jul 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@schlessera
schlessera force-pushed the update-framework branch 2 times, most recently from 38591cd to 14e9cc2 Compare July 21, 2026 17:03

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/update-framework.yml:
- Line 76: Update the workflow’s shell commands to avoid direct interpolation of
github.event.repository.default_branch: expose the value through an environment
variable such as DEFAULT_BRANCH and reference "$DEFAULT_BRANCH" in the --base
argument for PR lookup, creation, and editing commands. Keep the existing branch
and PR behavior unchanged.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 27dccad0-3262-48a6-a133-118acfcbbfae

📥 Commits

Reviewing files that changed from the base of the PR and between 284528f and 3845ec1.

⛔ Files ignored due to path filters (1)
  • composer.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • .github/workflows/update-framework.yml

Comment thread .github/workflows/update-framework.yml
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:framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants