Skip to content
This repository has been archived by the owner on Nov 4, 2024. It is now read-only.

Adding cargo-edit and gh to ci-unified #651

Merged
merged 3 commits into from
Aug 15, 2024
Merged

Conversation

mutantcornholio
Copy link
Contributor

This allows Cargo.toml manipulation in CI, primarily, versioning.
Will be useful for publishing crates, based on tags.

@mutantcornholio mutantcornholio changed the title Adding cargo-edit to unified-ci Adding cargo-edit to ci-unified Aug 14, 2024
@mutantcornholio mutantcornholio marked this pull request as draft August 15, 2024 08:59
This allows Cargo.toml manipulation in CI, primarily, versioning.
Will be useful for publishing crates, based on tags.
This currently fails to compile with `duplicate lang item in crate
`std` (which `memchr` depends on): `panic_impl`.`
We're also downloading a binary of this package just a bit earlier in
Dockerfile.
@mutantcornholio mutantcornholio force-pushed the yuri/add-cargo-edit branch 2 times, most recently from 5d25e34 to d8925b9 Compare August 15, 2024 10:37
@mutantcornholio mutantcornholio marked this pull request as ready for review August 15, 2024 15:21
@mutantcornholio mutantcornholio changed the title Adding cargo-edit to ci-unified Adding cargo-edit and gh to ci-unified Aug 15, 2024
@@ -83,6 +83,9 @@ RUN pip install yq

### generic ci ####

RUN wget https://github.com/cli/cli/releases/download/v2.54.0/gh_2.54.0_linux_amd64.deb && \
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it worth moving the 2.54.0 version to an ARG

@alvicsam alvicsam merged commit 38dc90f into master Aug 15, 2024
3 checks passed
@alvicsam alvicsam deleted the yuri/add-cargo-edit branch August 15, 2024 15:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants