Skip to content

Commit

Permalink
Fix the association of code coverage results with source code (opense…
Browse files Browse the repository at this point in the history
…arch-project#3668)

### Description
Fix the association of code coverage results with source code. You'll
have seen how this was broken because we stopped getting the comment on
PRs with coverage information. My apologies for breaking this in the
most recent change to centralized the CC runs.

### Issues Resolved
- Resolves opensearch-project#2649

### Testing
✅ Inspection CC report on this PR, see
[[link]](opensearch-project#3668 (comment)).

### Check List
- [ ] ~New functionality includes testing~
- [ ] ~New functionality has been documented~
- [X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and
signing off your commits, please check
[here](https://github.com/opensearch-project/OpenSearch/blob/main/CONTRIBUTING.md#developer-certificate-of-origin).

---------

Signed-off-by: Peter Nied <[email protected]>
  • Loading branch information
peternied authored Nov 10, 2023
1 parent d3e6efe commit b504ca4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,7 @@ jobs:
- "integration-tests"
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/download-artifact@v3
with:
path: downloaded-artifacts
Expand Down Expand Up @@ -101,7 +102,6 @@ jobs:
with: |
token: ${{ secrets.CODECOV_TOKEN }}
fail_ci_if_error: true
directory: downloaded-artifacts
verbose: true
Expand Down

0 comments on commit b504ca4

Please sign in to comment.