Skip to content

Conversation

h7x4
Copy link
Collaborator

@h7x4 h7x4 commented Sep 1, 2023

The pipeline has been updated with the following changes:

  • Run all steps in parallell
  • Update several of the steps, also the entirety of actions-rs seems to be unmaintained
  • Only run coveralls on PRs, to avoid breaking workflows on forks
  • Add dependency cache to speed up the builds.
  • Remove the benchmarking step, we have no benchmarks

We're still using action-rs/grcov which might break at some point as it's using deprecated API functions. I haven't found any good alternatives for this yet. But I think this should at least make less things break whenever the API gets deprecated for good.

h7x4 added 5 commits September 1, 2023 13:53
- `actions/checkout@v2` -> `actions/checkout@v3`
- `actions-rs/toolchain@v1` -> `dtolnay/rust-toolchain@nightly`
We don't have any benchmarks at the moment,
so this step is pointless
@h7x4 h7x4 changed the title Update main workflow, several changes Update main workflow with several changes Sep 1, 2023
@h7x4 h7x4 mentioned this pull request Sep 6, 2023
@h7x4 h7x4 requested a review from jiegec October 29, 2023 12:20
@jiegec jiegec merged commit f567e8d into jiegec:master Oct 29, 2023
@coveralls
Copy link

coveralls commented Jul 14, 2024

Pull Request Test Coverage Report for Build 6049149731

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-24.1%) to 41.79%

Totals Coverage Status
Change from base Build 5988722268: -24.1%
Covered Lines: 1527
Relevant Lines: 3654

💛 - Coveralls

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.

3 participants