Skip to content

Improve deploy workflow and distribution configuration#271

Draft
shadoath wants to merge 1 commit intomasterfrom
claude/debug-wordpress-deploy-action-LhyQL
Draft

Improve deploy workflow and distribution configuration#271
shadoath wants to merge 1 commit intomasterfrom
claude/debug-wordpress-deploy-action-LhyQL

Conversation

@shadoath
Copy link
Collaborator

Summary

This PR refines the WordPress.org deployment workflow and distribution package configuration to improve reliability and clarity.

Key Changes

Deploy Workflow (deploy.yml)

  • Tag filtering: Changed tag trigger from wildcard (*) to semantic versioning pattern ([0-9]+.[0-9]+.[0-9]+) to only deploy on valid version tags
  • Dry-run input: Converted from string-based to proper boolean type in workflow inputs
    • Changed default: 'true' to default: true (boolean)
    • Updated input reference from inputs['dry-run'] == 'true' to inputs.dry-run for cleaner syntax
    • This ensures the dry-run parameter is properly typed and handled as a boolean throughout the workflow

Distribution Configuration (.distignore)

  • Added /.distignore to the ignore list to exclude the distribution configuration file itself from packages
  • Added /CLAUDE.md to exclude development/documentation files from distribution

Implementation Details

The workflow improvements ensure that:

  1. Only properly versioned releases trigger automatic deployment
  2. The dry-run parameter uses native GitHub Actions boolean type handling instead of string comparison
  3. Distribution packages are cleaner by excluding configuration and development files

https://claude.ai/code/session_01DM75vWFtVxixaWhENsBT22

…t, update .distignore

- Restrict tag pattern to version-like tags ([0-9]+.[0-9]+.[0-9]+) to prevent
  accidental deployments from non-release tags
- Add type: boolean to dry-run workflow_dispatch input for proper checkbox UI
  and simplify the comparison expression
- Add CLAUDE.md and .distignore to .distignore so dev-only files are excluded
  from the WordPress.org SVN deployment

https://claude.ai/code/session_01DM75vWFtVxixaWhENsBT22
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.

2 participants