Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Introduce a script and a Github workflow to ensure our WooCommerce templates are always up to date with the latest version. #847

Merged
merged 5 commits into from
Mar 13, 2023

Conversation

eherman-godaddy
Copy link
Contributor

@eherman-godaddy eherman-godaddy commented Mar 8, 2023

Introduce a way to automate the process of keeping the WooCommerce cart.php template files version, that we bundle, up to date. This script reaches out to the WooCommerce template file on Github and compares our bundled template version to the most up to date one. If the version on the WooCommerce github is greater than the one we bundle, our Github workflow will bump the version in the bundled template and commit it back to Github, ensuring that our template version always matches the most up to date one on Github.

  • The script can be run locally with npm run update:woo-template-versions.
  • The script will run on any commits into master via this workflow, update the template file (if needed) and then commit the updated file back to Github.

@eherman-godaddy eherman-godaddy self-assigned this Mar 8, 2023
@eherman-godaddy eherman-godaddy added the [Type] Woo Relevant to WooCommerce label Mar 8, 2023
@eherman-godaddy eherman-godaddy added this to the 1.8.1 milestone Mar 8, 2023
@eherman-godaddy eherman-godaddy changed the title Bump cart template version to 7.4.0 Introduce a script and a Github workflow to ensure our WooCommerce templates are always up to date with the latest version. Mar 8, 2023
.dev/bin/update-woo-template-versions.php Outdated Show resolved Hide resolved
.dev/bin/update-woo-template-versions.php Outdated Show resolved Hide resolved
@jbardo-godaddy
Copy link
Member

@eherman-godaddy Brilliant! Good on you to takle that

@jbardo-godaddy
Copy link
Member

@eherman-godaddy anyway you think of we could run this on Github action instead of locally?

@EvanHerman
Copy link
Contributor

@jbardo-godaddy
Copy link
Member

@jbardo-godaddy Yeah, this whole workflow should only run on any changes to master https://github.com/godaddy-wordpress/go/pull/847/files#diff-c8ffdf393c0c7fca38f0c86f5ee054bdc4f57947b291bdc5fcfc25a5998a50b5R1-R40

Sorry I miss that. LGTM then

@jbardo-godaddy jbardo-godaddy merged commit fbe76ab into master Mar 13, 2023
@jbardo-godaddy jbardo-godaddy deleted the update-cart-template-version branch March 13, 2023 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Woo Relevant to WooCommerce
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants