Skip to content

ci: pick plugins to sync from the release-please manifest - #828

Merged
josephfusco merged 1 commit into
mainfrom
ci/sync-from-manifest
Oct 23, 2026
Merged

josephfusco merged 1 commit into
mainfrom
ci/sync-from-manifest

Conversation

@josephfusco

Copy link
Copy Markdown
Member

Follow-up to #820. The sync job chose which plugins to update by diffing plugins/*/package.json. That misses a plugin whose package.json already holds the new version.

That's what happened on #825 (webhooks 0.0.8). package.json was bumped to 0.0.8 under changesets but never released, so release-please had nothing to change there. As a result:

  • the sync job skipped webhooks
  • composer.json and the plugin header stayed on 0.0.7
  • composer-packages.json got no 0.0.8 entry

Every release PR bumps .release-please-manifest.json, so the job now syncs the plugins whose manifest version differs from the merge base.

Tested the jq against #825's branch locally: it returns plugins/wpgraphql-webhooks.

Use of AI Tools

AI assistance: Yes
Tool(s): Claude Code
Model(s): Claude Opus 5.5
Used for: Assisting with diagnosing the missed sync on #825 and fixing plugin detection.

@josephfusco
josephfusco requested review from a team and colinmurphy as code owners October 22, 2026 23:31
@changeset-bot

changeset-bot Bot commented Oct 22, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 25229c4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@josephfusco
josephfusco merged commit 5fa27d9 into main Oct 23, 2026
2 checks passed
@josephfusco
josephfusco deleted the ci/sync-from-manifest branch October 23, 2026 14:56
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.

1 participant