Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: relax integration checks #1339

Merged
merged 2 commits into from
Jun 17, 2024
Merged

Conversation

cdcabrera
Copy link
Member

@cdcabrera cdcabrera commented Jun 17, 2024

What's included

  • ci: relax integration checks

Notes

  • the integration checks seem to suffer from the same setback here too...
    • the env setup creates build output differences which in turn creates failures in the output checks. since we're already performing these checks in multiple places lets remove/relax them here too

How to test

  • Confirm the related ci check now passes.
    • There may still be failure, you'll have to review the related console output to confirm

Example

...

Updates issue/story

ongoing
related #1290

@cdcabrera cdcabrera added build 202408 project phase labels Jun 17, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jun 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.99%. Comparing base (a065d98) to head (5f8d7ee).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1339   +/-   ##
=======================================
  Coverage   91.99%   91.99%           
=======================================
  Files         113      113           
  Lines        4399     4399           
  Branches     1857     1857           
=======================================
  Hits         4047     4047           
  Misses        332      332           
  Partials       20       20           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a065d98...5f8d7ee. Read the comment docs.

@cdcabrera cdcabrera merged commit c83b2c4 into RedHatInsights:main Jun 17, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants