mageos-magento2 icon indicating copy to clipboard operation
mageos-magento2 copied to clipboard

Write a Github Action that runs the the semantic version checker as defined by https://developer.adobe.com/commerce/testing/guide/svc/

Open damienwebdev opened this issue 3 years ago • 8 comments
trafficstars

damienwebdev avatar Oct 28 '22 19:10 damienwebdev

So, the semver checker tool wants to compare whatever changes you have against a known version or tag. I don't see a way to have it auto-determine which version your new change or branch should validate against, so should I just hard-code it to 2.4-develop for now, and we'll just bump it manually as new (or old) versions are added?

From the instructions:

  1. Navigate the magento-semver folder and run the Semantic Version Checker compare command: bin/svc compare ../magento2-mainline ../magento2 The first parameter is the mainline code without any changes, and the second parameter is the path to the folder with your changes. The results of the Semantic Version Checker are outputted to the console.

lefte avatar Oct 31 '22 20:10 lefte

Unrelated question: Check semver on every push, on pull requests, what trigger do we want to use?

lefte avatar Oct 31 '22 21:10 lefte

On PRs only currently, comparing the diff of the PR against the original HEAD.

damienwebdev avatar Oct 31 '22 21:10 damienwebdev

MageOS Semantic Checker

See the following potential pull request into the MageOS repo: https://github.com/mage-os/mageos-magento2/compare/2.4-develop...lefte:magento2:feature/issue-5-github-action-semantic-checker

Would this be the right spot to create a PR to?

lefte avatar Nov 01 '22 16:11 lefte

On PRs only currently, comparing the diff of the PR against the original HEAD.

Hopefully this is what you meant, it compares the new pull request's head branch against the base branch

lefte avatar Nov 01 '22 16:11 lefte

@lefte this is exactly it! Thank you so so so much!

damienwebdev avatar Nov 01 '22 16:11 damienwebdev

🚯 Ah, I see it doesn't like pull requests from forked repositories, let me check on which variable to use for that

lefte avatar Nov 02 '22 01:11 lefte

Alright, I believe the newest action https://github.com/mage-os/mageos-magento2/actions/runs/3374600139 should now work with forked repositories. Thanks for your patience.

lefte avatar Nov 02 '22 03:11 lefte