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

[CI] fix error in release step that generates checksums #4374

Merged
merged 1 commit into from Feb 14, 2024

Conversation

wileyj
Copy link
Contributor

@wileyj wileyj commented Feb 13, 2024

During workflow testing a release in a downstream fork, an unexpected error kept surfacing:

Poetry could not find a pyproject.toml file in /github/workspace or its parents

https://github.com/wileyj/stacks-blockchain/actions/runs/7878083852/job/21495965471#step:4:11

updating this specific action's commit to the latest release v1.0.1 resolves the immediate issue.
we'll be moving this workflow out of this repo (possibly refactoring it using bash or something, since it's a simple task that really shouldn't fail due to a python dependency).

@wileyj
Copy link
Contributor Author

wileyj commented Feb 13, 2024

Note: ci will not run here without a code change, since it's only a small change to a yml file.
that's a different discussion we should have about how and when to trigger CI (currently, it's configured to not run on changes to markdown or yaml)

@jbencin
Copy link
Contributor

jbencin commented Feb 13, 2024

we'll be moving this workflow out of this repo

I was going to suggest this, sha512sum available in pretty much every command line environment

@wileyj
Copy link
Contributor Author

wileyj commented Feb 14, 2024

we'll be moving this workflow out of this repo

I was going to suggest this, sha512sum available in pretty much every command line environment

thanks! i did open an issue for precisely this #4375

@jcnelson jcnelson merged commit 021eaae into master Feb 14, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants