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

Avoid running publish-go-tester-task if tag is set #295

Merged
merged 1 commit into from
Dec 19, 2022

Conversation

MatteoPologruto
Copy link
Contributor

publish-go-tester-task runs when a tag is pushed. Since the tag has priority when version is defined, the workflow fails because it does not find any build named test-XXX-git-snapshot when calculating the checksums. It recently happened here: https://github.com/arduino/arduino-cli/actions/runs/3480562942/jobs/5820576240#step:3:16
To fix this issue, during run-determination, the presence of a tag is checked comparing github.ref with TAG_REGEX. If there is a match, the other jobs are skipped, preventing the workflow from failing at later stages.

@MatteoPologruto MatteoPologruto added type: imperfection Perceived defect in any part of project topic: code Related to content of the project itself labels Dec 7, 2022
@MatteoPologruto MatteoPologruto self-assigned this Dec 7, 2022
workflow-templates/publish-go-tester-task.yml Outdated Show resolved Hide resolved
workflow-templates/publish-go-tester-task.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

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

Thanks Matteo!

@per1234
Copy link
Contributor

per1234 commented Dec 16, 2022

@MatteoPologruto I see the merge is blocked because the required spellcheck workflow job run didn't trigger on the last push. I have no idea why it didn't run. It is likely just a transient glitch in GitHub Actions.

Please let me know if you are ready to merge this and I'll take care of overriding the requirement and merging it (you have the permissions to do it also, but I already am familiar with what needs to be done so it will be most efficient for me to take care of it in this case).

@MatteoPologruto
Copy link
Contributor Author

Hi @per1234, I'm ready to merge. I could also try to re-trigger a spellcheck workflow run by force pushing the same commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants