Skip to content

ci: Remove superfluous worfklow#34

Merged
tlater-famedly merged 2 commits intomainfrom
tlater/remove-rust
Feb 17, 2025
Merged

ci: Remove superfluous worfklow#34
tlater-famedly merged 2 commits intomainfrom
tlater/remove-rust

Conversation

@tlater-famedly
Copy link
Copy Markdown
Contributor

Over time, this workflow has become a slightly less featureful version of the rust-workflow. This means we're unnecessarily running a bunch of stuff in CI twice.

@tlater-famedly tlater-famedly requested a review from a team as a code owner February 10, 2025 04:09
@tlater-famedly tlater-famedly force-pushed the tlater/remove-rust branch 4 times, most recently from 9c97d2a to 0a9d117 Compare February 10, 2025 04:26
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 10, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Over time, this workflow has become a slightly less featureful version
of the rust-workflow. This means we're unnecessarily running a bunch
of stuff in CI twice.
The nextest installed in our docker containers now fails by default
when no tests are found.
Copy link
Copy Markdown

@Revanee Revanee left a comment

Choose a reason for hiding this comment

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

LGTM

@tlater-famedly tlater-famedly merged commit 5199f17 into main Feb 17, 2025
1 check passed
@tlater-famedly tlater-famedly deleted the tlater/remove-rust branch February 17, 2025 03:28
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