Skip to content

Conversation

@mike-day
Copy link
Member

@mike-day mike-day commented Apr 24, 2025

JIRA Ticket

CICD-892

What Are We Doing Here

A new mu-plugin should be ignored in site deploys. We can do so by adding the plugin dir to our excludes script.

Testing

Unit

Ran the make target and confirmed all unit tests pass with the updated fixtures.

Manual

In a local test site, I ran through the following steps:

  • Created an mu-plugin under a new wpe-update-source-selector dir
  • Changed dirs to wp-content (problematic for mu-plugin excludes in the past)
  • Created this .env file (note the --dry-run flag):
    CICD-892_env-file
  • Using the new image, ran the dry run deploy and noted the file list:
    CICD-892_deploy-file-list
  • Updated the .env to remove the --dry-run flag
  • Ran a real deploy, confirming the file list still excluded the new mu-plugin

@mike-day mike-day marked this pull request as ready for review April 25, 2025 03:44
@mike-day mike-day requested a review from a team as a code owner April 25, 2025 03:44
@mike-day mike-day merged commit c101fd0 into main Apr 25, 2025
2 checks passed
@mike-day mike-day deleted the CICD-892-ignore-wpe-update-source-selector branch April 25, 2025 13:24
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.

3 participants