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

chore(ci): make sure to only make new releases if tests pass #764

Merged
merged 4 commits into from
Feb 17, 2021

Conversation

SimenB
Copy link
Member

@SimenB SimenB commented Feb 17, 2021

https://github.com/jest-community/eslint-plugin-jest/runs/1914618070?check_suite_focus=true made a release even though tests were failing. #689 obviously didn't work, so let's bloat up a single file instead

I also added a prepare-yarn-cache job which ensures all other jobs have an updated cache before running

@SimenB
Copy link
Member Author

SimenB commented Feb 17, 2021

Things happened in the correct order here - prepare-cache ran first, and only when that completed did all the tests runs spawn. The docs job also ran (it's supposed to only run on PRs, so will verify it does not run on master)

And release was skipped, but only showed up after the other jobs completed

@SimenB
Copy link
Member Author

SimenB commented Feb 17, 2021

a failing test made no difference, it only said "skipped" still. Let's trust it works 😛 https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#jobsjob_idneeds

@G-Rath
Copy link
Collaborator

G-Rath commented Feb 17, 2021

Sorry, I didn't realize we'd switched up CI like that - that explains why it released.

I played with the workflow event myself last month and couldn't find a way to have it only run on success

@SimenB
Copy link
Member Author

SimenB commented Feb 17, 2021

Switched up? We've been using GH actions since Oct 2019 to make releases. And if you meant changes in #689 you did comment on them 😛

@SimenB SimenB merged commit b1d6111 into master Feb 17, 2021
@SimenB SimenB deleted the fix-ci-release branch February 17, 2021 08:14
@G-Rath
Copy link
Collaborator

G-Rath commented Feb 17, 2021

I meant #689 - the only comment j made in that PR was just now :)

(Unless the mobile UI is bidding previous comments?)

Still, no harm no foul 🤷‍♂️

@SimenB
Copy link
Member Author

SimenB commented Feb 17, 2021

Regardless, seems fixed now
image

@SimenB
Copy link
Member Author

SimenB commented Feb 17, 2021

I've also deprecated the broken release

image

https://www.npmjs.com/package/eslint-plugin-jest/v/24.1.4

@G-Rath
Copy link
Collaborator

G-Rath commented Feb 17, 2021

Cheers - sorry again for the hassle 😓

@SimenB
Copy link
Member Author

SimenB commented Feb 17, 2021

No worries, I'm the one who set up the broken automation 😀

@G-Rath
Copy link
Collaborator

G-Rath commented Feb 17, 2021

Btw while I've got you, would you be good to turn on automatic deleting of head branches?

@SimenB
Copy link
Member Author

SimenB commented Feb 17, 2021

Nice, didn't know GH had added that! I guess that happens automatically for me since I use refined GH.

Turned on both of these bad boys

image

@github-actions
Copy link

github-actions bot commented Mar 6, 2021

🎉 This PR is included in version 24.1.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants