Skip to content

Allow not-yet-effective successes in output#939

Merged
mbestavros merged 1 commit into
conforma:mainfrom
mbestavros:hacbs-2449
Aug 15, 2023
Merged

Allow not-yet-effective successes in output#939
mbestavros merged 1 commit into
conforma:mainfrom
mbestavros:hacbs-2449

Conversation

@mbestavros

@mbestavros mbestavros commented Aug 15, 2023

Copy link
Copy Markdown

Pretty simple patch. Maybe we want to consider some additional annotation to indicate that this rule isn't yet effective?

Ref: https://issues.redhat.com/browse/HACBS-2449

Signed-off-by: Mark Bestavros <mbestavr@redhat.com>
@codecov

codecov Bot commented Aug 15, 2023

Copy link
Copy Markdown

Codecov Report

Merging #939 (9803e85) into main (9e92878) will decrease coverage by 0.03%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #939      +/-   ##
==========================================
- Coverage   85.57%   85.54%   -0.03%     
==========================================
  Files          64       64              
  Lines        5178     5175       -3     
==========================================
- Hits         4431     4427       -4     
- Misses        747      748       +1     
Flag Coverage Δ
acceptance 69.36% <ø> (+0.02%) ⬆️
generative ?
integration 62.67% <ø> (ø)
unit 71.68% <ø> (+0.02%) ⬆️

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

Files Changed Coverage Δ
internal/evaluator/conftest_evaluator.go 85.17% <ø> (+0.26%) ⬆️

... and 1 file with indirect coverage changes

@robnester-rh robnester-rh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

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