Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[major] Refactor WP install scripts to use WP-CLI #3

Merged
merged 145 commits into from
Feb 13, 2024

Conversation

jazzsequence
Copy link
Contributor

@jazzsequence jazzsequence commented Feb 8, 2024

This PR refactors the scripts to no longer use the community-contributed install-wp-tests.sh script.

Functions have been broken out into a helpers.sh file that is sourced where needed.

Tests have been updated and additional tests that run through the actual scripts have been added to this repository. This is a major change since there are fundamental improvements made to how the scripts even work.

Additionally, improvements have been made to the internal tests as a whole, pulling complex scripting out of the test.yml file and into their own scripts in .github/workflows/bin.

Pulled out of pantheon-systems/pantheon-mu-plugin#29 so the scripts can be centralized and updated in one place rather than using copy/pasta.

Successful run using this package: https://github.com/pantheon-systems/pantheon-mu-plugin/actions/runs/7836581235/job/21422397789

@jazzsequence jazzsequence changed the title Add more helpers Add helpers Feb 8, 2024
@jazzsequence jazzsequence marked this pull request as ready for review February 8, 2024 22:04
@jazzsequence jazzsequence requested a review from a team as a code owner February 8, 2024 22:04
Copy link
Member

@pwtyler pwtyler left a comment

Choose a reason for hiding this comment

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

these were written before our discussion earlier but posting all the same

@jazzsequence jazzsequence requested a review from pwtyler February 13, 2024 16:43
@jazzsequence jazzsequence merged commit 8f77f20 into main Feb 13, 2024
2 checks passed
@jazzsequence jazzsequence deleted the add-more-helpers branch February 13, 2024 21:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants