-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
fix: broken unpublish workflow #2365
Conversation
Coverage Report:
Changed Files:Coverage values did not change👌. |
@@ -15,7 +15,7 @@ concurrency: | |||
jobs: | |||
clean-npm-versions: | |||
name: Unpublish versions next and pr | |||
runs-on: buildjet-4vcpu-ubuntu-2204 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@danielbate what was the impetus for running on this specific hardware?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe it was lifted from the frontend codebases (e.g. fuel-connectors)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This hardware should be superior and have higher availability.
Why are we rolling this config back?
This job didn't actually succeed. I think it'll be worth refactoring |
Marking as a draft until we are sure about the script itself |
Closing in favour of issue #2377 |
The workflow responsible for unpublishing old versions/tags from npm was broken.
This PR fixes that.
Successful run on this branch: https://github.com/FuelLabs/fuels-ts/actions/runs/9179019639/job/25240208986