Skip to content

ci: fix CI badge URL → JuliaData + add codecov.yml#71

Merged
mathieu17g merged 1 commit into
mainfrom
ci-coverage-visibility
Jun 21, 2026
Merged

ci: fix CI badge URL → JuliaData + add codecov.yml#71
mathieu17g merged 1 commit into
mainfrom
ci-coverage-visibility

Conversation

@mathieu17g

Copy link
Copy Markdown
Collaborator
  • README: point the CI badge at JuliaData/XML.jl — it still pointed at JuliaComputing (working only via redirect since the move).
  • .github/codecov.yml: coverage reported as informational (never blocks CI) + a PR comment when coverage changes.

A coverage badge is deferred to a follow-up: Codecov still tracks the package under the old JuliaComputing path (63%) and has no data under JuliaData, so a badge would render "unknown" until that's reconciled — see below.

@ViralBShah — two Codecov items need admin:

  1. Activate the repo on Codecov under JuliaData — it still tracks JuliaComputing/XML.jl (63% coverage); JuliaData/XML.jl has no data, so uploads/badges don't land there.
  2. Set a CODECOV_TOKEN repo secret — CI passes no token to codecov-action@v7, so uploads can intermittently rate-limit (the step still exits green, masking it).

With both, uploads land under JuliaData and a coverage badge will render a real number.

- README: point the CI badge at JuliaData (it still pointed at JuliaComputing,
  working only via redirect since the org move).
- .github/codecov.yml: report coverage as informational (never blocks CI) +
  comment on PRs that change coverage.

A coverage badge is deferred: Codecov still tracks the package under the old
JuliaComputing path (63%) and shows nothing under JuliaData, so a badge would
render 'unknown' until the move is reconciled on Codecov.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ViralBShah

Copy link
Copy Markdown
Contributor

You should have admin now.

@mathieu17g

Copy link
Copy Markdown
Collaborator Author

Partial solution to #73 — this covers the two tasks that don't need Codecov-side access:

  • the CI badge now points at JuliaData (it still targeted JuliaComputing, working only via redirect);
  • .github/codecov.yml adds informational status checks + the PR-comment layout for patch & project coverage.

The remaining work (Codecov GitHub-App access + upload token, then the CODECOV_TOKEN secret + CI.yml wiring + the badge) stays tracked in #73, blocked on the org-owner Codecov items. Merging this does not close #73.

@mathieu17g mathieu17g merged commit b40e56b into main Jun 21, 2026
8 checks passed
@mathieu17g mathieu17g deleted the ci-coverage-visibility branch June 21, 2026 07:14
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