Skip to content

ci: check that benchmarks compile#1828

Merged
mattsse merged 2 commits intomainfrom
onbjerg/bench-check
Mar 18, 2023
Merged

ci: check that benchmarks compile#1828
mattsse merged 2 commits intomainfrom
onbjerg/bench-check

Conversation

@onbjerg
Copy link
Collaborator

@onbjerg onbjerg commented Mar 18, 2023

Closes #1762

@onbjerg onbjerg added C-enhancement New feature or request C-test A change that impacts how or what we test labels Mar 18, 2023
@onbjerg onbjerg requested a review from gakonst as a code owner March 18, 2023 06:38
@codecov-commenter
Copy link

codecov-commenter commented Mar 18, 2023

Codecov Report

Merging #1828 (f402b5f) into main (d774d91) will decrease coverage by 0.05%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #1828      +/-   ##
==========================================
- Coverage   73.53%   73.49%   -0.05%     
==========================================
  Files         410      410              
  Lines       50510    50510              
==========================================
- Hits        37144    37120      -24     
- Misses      13366    13390      +24     
Flag Coverage Δ
integration-tests 19.68% <ø> (ø)
unit-tests 67.84% <ø> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 7 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

nice!

@mattsse mattsse merged commit 661a242 into main Mar 18, 2023
@mattsse mattsse deleted the onbjerg/bench-check branch March 18, 2023 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-enhancement New feature or request C-test A change that impacts how or what we test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cover bench files in CI

3 participants