Skip to content

ci: various cleanups#139

Open
nbdd0121 wants to merge 6 commits intomainfrom
dev/ci
Open

ci: various cleanups#139
nbdd0121 wants to merge 6 commits intomainfrom
dev/ci

Conversation

@nbdd0121
Copy link
Copy Markdown
Member

No description provided.

Yaml language server wants to format strings with double quotes.

Signed-off-by: Gary Guo <gary@garyguo.net>
These checks take longer time to setup than actual checking. Even though
we're not paying for these CI runs, let's not waste energy in downloading
toolchain repeatedly.

Lump them into a single job.

Signed-off-by: Gary Guo <gary@garyguo.net>
check is already covered by test, so no need to run both jobs.

Signed-off-by: Gary Guo <gary@garyguo.net>
The most time-consuming part of cargo-rdme is the cargo-install. Use
cache-cargo-install-action to cache it, so we can consolidate it similar to
other fast checks.

Signed-off-by: Gary Guo <gary@garyguo.net>
Significant amount of time in the CI jobs are building cargo tool from
source. Install them from releases instead.

Signed-off-by: Gary Guo <gary@garyguo.net>
When a pull request is updated, cancel old CI runs.

Signed-off-by: Gary Guo <gary@garyguo.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant