Skip to content

Remove Codecov upload step - #150

Merged
v1r3n merged 1 commit into
mainfrom
ci/disable-codecov-comment
Aug 6, 2026
Merged

Remove Codecov upload step#150
v1r3n merged 1 commit into
mainfrom
ci/disable-codecov-comment

Conversation

@kowser-orkes

Copy link
Copy Markdown
Contributor

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • WHOSUSING.md
  • Other (please describe):

Changes in this PR

  • remove the Codecov upload step from the build job
  • jacoco report still generated, unaffected
  • Publish Test Report check (mikepenz/action-junit-report) already covers pass/fail counts, unaffected

Alternatives considered

  • silence just the PR comment and keep the upload — dropped, the App isn't installed so there's nothing else the upload does

- comment was the only visible signal, no status check, no App installed
- jacoco report still generated, Publish Test Report check covers test results
@kowser-orkes
kowser-orkes force-pushed the ci/disable-codecov-comment branch from 526ff32 to 86441a7 Compare August 6, 2026 06:05
@v1r3n
v1r3n merged commit db140f9 into main Aug 6, 2026
9 checks passed
@v1r3n
v1r3n deleted the ci/disable-codecov-comment branch August 6, 2026 07:23
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.

2 participants