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] Bump version to 0.4.0 and fix release action #224

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

zeegomo
Copy link
Contributor

@zeegomo zeegomo commented Apr 7, 2022

Bump version and fix some problems in the release action that prevented it from completing successfully

zeegomo and others added 3 commits April 7, 2022 11:12
Remove windows-gnu as it's not tested on PRs, not built for on
Jormungandr or catalyst-toolbox and is currently failing
@zeegomo zeegomo requested a review from a team April 7, 2022 09:18
Comment on lines +167 to +168
[profile.release]
lto = "thin"
Copy link
Contributor

Choose a reason for hiding this comment

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

we could also just add this in the Cargo.toml, is it better to just do it here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tbh I just picked the commit we had on jormungandr

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

Successfully merging this pull request may close these issues.

2 participants