Skip to content

Make coverage upload non-blocking in CI#9

Closed
Copilot wants to merge 7 commits into
mainfrom
copilot/fix-lint-and-test-with-cov
Closed

Make coverage upload non-blocking in CI#9
Copilot wants to merge 7 commits into
mainfrom
copilot/fix-lint-and-test-with-cov

Conversation

Copilot AI commented Jun 5, 2026

Copy link
Copy Markdown

The coverage stage was failing even when tests and coverage generation succeeded, due to intermittent/unsupported actions/upload-code-coverage API responses (HTTP 404). This change keeps coverage upload enabled while preventing upload failures from failing the lint-and-test-with-cov job.

  • Coverage failure mode

    • The failing step was the coverage upload action, not test execution or coverage.xml generation.
  • Workflow behavior update

    • Updated .github/workflows/python-package.yaml to make the upload step best-effort by setting fail-on-error: false on actions/upload-code-coverage@v1.2.0.
    • Existing permission and event guards remain unchanged.
- uses: actions/upload-code-coverage@v1.2.0
  with:
    fail-on-error: false
    file: coverage.xml
    language: Python
    label: code-coverage/pytest

@jonathandung

Copy link
Copy Markdown
Owner

@copilot also note that the GraalPy and Ubuntu tests with Python 3.12 appear to be hanging. The main issue with the failing check lies in the upload code coverage step native to GitHub, which was just recently rolled out.

Copilot AI changed the title [WIP] Fix failing GitHub Actions job lint-and-test-with-cov Fix lint-and-test-with-cov failure by correcting coverage upload permissions Jun 5, 2026
Copilot AI requested a review from jonathandung June 5, 2026 13:06
Copilot AI changed the title Fix lint-and-test-with-cov failure by correcting coverage upload permissions Make coverage upload non-blocking in CI Jun 5, 2026
@jonathandung

Copy link
Copy Markdown
Owner

@copilot isn't contents: read the default though?

@jonathandung jonathandung deleted the copilot/fix-lint-and-test-with-cov branch June 5, 2026 13:25
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