-
Notifications
You must be signed in to change notification settings - Fork 30
Code-coverage self-contained machinery, phase 1 #388
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
Open
hemidactylus
wants to merge
1
commit into
main
Choose a base branch
from
SL-codecov0
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+660
−487
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
17a7333 to
735277b
Compare
most (local) test coverage/reporting in place draft for codecov orchestrator TEMP reduce it scope for testing codecov relative path of workflows, fix workflows called with full syntax nope, revert to abs path and nothing more standardize yaml naming standardize yaml naming 2 secret inheritance chain overhaul add env to unit test job add env block to all testing flows trying to debug uploader: trying to debug uploader 2 trying to debug uploader 3 trying to debug uploader 4 trying to debug uploader 5 remove debug what is in the downloaded things anyway? coverage calculation and posting to PR, attempt 1 coverage calculation and posting to PR, attempt 2 coverage calculation and posting to PR, attempt 3 coverage calculation and posting to PR, attempt 4 coverage calculation and posting to PR, attempt 5 coverage calculation and posting to PR, attempt 6 coverage calculation and posting to PR, attempt 7 coverage calculation and posting to PR, attempt 8 publish html codecov report publish html codecov report 2 publish html codecov report 3 publish html codecov report 4 remove and postpone the 'gh pages html publish' thing minor coverage message rephrase html report retention workaround 1 html report retention workaround 2 reinstate full test; coverage json uploaded for push-to-main suspend farr-related tests on HCD until dev rer.serv. back in action revert the exclusion of farr in hcd tests, i.e. include them again aggregator workflow cleanup 2
bd57ccc to
4dd9714
Compare
Coverage reportfor commit: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Phase 1 means that this PR refactors the whole unit+integration test workflows in the following way:
Phase 2 will add a step, in PRs, that compares the currently-generated JSON report to that of the base commit (i.e. on main) to post a comment with delta information.