Skip to content

Commit

Permalink
fix: remove IT test coverage publishing to CodeCov (#2456)
Browse files Browse the repository at this point in the history
## Proposed change

disable temporary IT test coverage publishing to CodeCov.

<!--
Please include a summary of the changes and the related issue.
Please also include relevant motivation and context.
-->

## Related issues

<!--
Please make sure to follow the [contribution
guidelines](https://github.com/amadeus-digital/Otter/blob/main/CONTRIBUTING.md)
-->

* 🐛 related to #2455
<!-- * 🐛 Fix resolves #issue -->
<!-- * 🚀 Feature #issue -->
<!-- * 🚀 Feature resolves #issue -->
<!-- * :octocat: Pull Request #issue -->
  • Loading branch information
kpanot authored Nov 13, 2024
2 parents e94ab3c + 526d5da commit 6453a3a
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
11 changes: 6 additions & 5 deletions .github/workflows/it-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -110,11 +110,12 @@ jobs:
- name: Test
id: it-tests
run: yarn test-int --collectCoverage
- name: Upload results to Codecov
uses: codecov/codecov-action@b9fd7d16f6d7d1b5d2bec1a2887e65ceed900238 # v4.6.0
with:
flags: integration
token: ${{ secrets.CODECOV_TOKEN }}
# TODO: re-activate IT coverage publishing when #2455 is fixed
# - name: Upload results to Codecov
# uses: codecov/codecov-action@b9fd7d16f6d7d1b5d2bec1a2887e65ceed900238 # v4.6.0
# with:
# flags: integration
# token: ${{ secrets.CODECOV_TOKEN }}
- name: Zip generated app on failure
if: failure() && steps.it-tests.conclusion == 'failure'
run: |
Expand Down
2 changes: 1 addition & 1 deletion codecov.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
codecov:
require_ci_to_pass: false
require_ci_to_pass: true

ignore:
# Package Manager files
Expand Down

0 comments on commit 6453a3a

Please sign in to comment.