Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@djc
Copy link
Member

@djc djc commented Dec 19, 2022

No description provided.

@djc djc requested a review from ctz December 19, 2022 09:37
@djc djc force-pushed the msrv branch 2 times, most recently from 7d97711 to b68e4c2 Compare December 19, 2022 10:00
Copy link
Member

@ctz ctz left a comment

Choose a reason for hiding this comment

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

lgtm

@ctz
Copy link
Member

ctz commented Dec 19, 2022

Side note: I've been quite impressed by cargo-llvm-cov so would like to slowly start replacing the various admin/coverage copies we have with straightforward uses of that instead. (That also improves the LLVM upgrade carousel, cos it's done for us by the rustup llvm-tools component.)

@djc
Copy link
Member Author

djc commented Dec 19, 2022

Yes, that sounds great!

@codecov-commenter
Copy link

Codecov Report

Merging #9 (1ae20e0) into main (ad85d49) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         4           
  Lines          282       280    -2     
=========================================
- Hits           282       280    -2     
Impacted Files Coverage Δ
src/tests.rs 100.00% <ø> (ø)
src/lib.rs 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@djc djc merged commit c169bae into main Dec 20, 2022
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.

4 participants