Skip to content

Conversation

@kasparsd
Copy link
Collaborator

What?

  • Ensure the blueprint.json is deployed to WP.org assets directory.
  • Use a dedicated deploy action for asset and readme changes to avoid tagging version releases for minor updates.

Why?

Fixes #731.

How?

Note:

  • This will deploy the readme.txt of the latest release tag version before the tag has been created since that happens after merging the release pull request into master. I'm not 100% sure what WP-org does when the specified stable tag is actually not part of SVN.

Testing Instructions

Screenshots or screencast

Changelog Entry

Added - New feature.
Changed - Existing functionality.
Deprecated - Soon-to-be removed feature.
Removed - Feature.
Fixed - Bug fix.
Security - Vulnerability.

if: github.ref_name == 'master'
env:
# Note: this action doesn't support BUILD_DIR so it pushes the raw readme.txt
IGNORE_OTHER_FILES: true
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This action is inconsistent in two ways

  1. It doesn't support BUILD_DIR so we can't point it to the correct directory of built files (according to our release logic).

  2. It attempts to deploy the un-built source files unless we specify IGNORE_OTHER_FILES which would deploy and invalid release.

- name: WordPress.org deploy
id: deploy
uses: 10up/action-wordpress-plugin-deploy@stable
if: startsWith( github.ref, 'refs/tags/' )
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This action derives the release VERSION from the tag name so it can't run anywhere else but on tag builds.

@kasparsd kasparsd merged commit 123558f into master Dec 11, 2025
6 of 54 checks passed
@kasparsd kasparsd deleted the fix-blueprint-and-wporg-asset-deploys branch December 11, 2025 10:08
@jeffpaul jeffpaul added this to the 0.15.0 milestone Dec 11, 2025
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.

Blueprint not working

3 participants