Skip to content

Codecov fixes#1289

Merged
fizyk merged 1 commit intomainfrom
codecov
Mar 1, 2026
Merged

Codecov fixes#1289
fizyk merged 1 commit intomainfrom
codecov

Conversation

@fizyk
Copy link
Copy Markdown
Member

@fizyk fizyk commented Mar 1, 2026

Chore that needs to be done:

  • Add newsfragment pipenv run towncrier create [issue_number].[type].rst

Types are defined in the pyproject.toml, issue_number either from issue tracker or the Pull request number

Summary by CodeRabbit

  • Chores
    • Updated continuous integration so coverage tooling credentials are supplied consistently across all test workflows and platforms; no changes to test behaviour or settings.
    • Added a news fragment noting a fix to the coverage pipeline configuration to reflect and communicate this CI update.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cc53804 and b1d68e2.

📒 Files selected for processing (2)
  • .github/workflows/tests.yml
  • newsfragments/+e9a4c914.misc.rst
✅ Files skipped from review due to trivial changes (1)
  • newsfragments/+e9a4c914.misc.rst
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/tests.yml

📝 Walkthrough

Walkthrough

The pull request adds the CODECOV_TOKEN secret to all test-related GitHub Actions jobs in .github/workflows/tests.yml and adds a news fragment noting the codecov pipeline configuration fix.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/tests.yml
Injected CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} into all test job secrets blocks (postgresql_* , macos_, docker_postgresql_) across versions 14–18; no other job parameters changed.
Release Notes
newsfragments/+e9a4c914.misc.rst
Added a brief news fragment: "Fix codecov pipeline configuration."

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 I hopped into workflows, quiet and quick,
Poked a secret where metrics would stick.
Now codecov sings through each test-run night,
Coverage lights gleam soft and bright.
Hop hop, the pipeline hums just right. 🐇

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Codecov fixes' directly relates to the main changes in the pull request, which involve adding CODECOV_TOKEN secrets to GitHub Actions workflows and creating a corresponding newsfragment noting the codecov pipeline configuration fix.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Newsfragment Check ✅ Passed Newsfragment file (+e9a4c914.misc.rst) added with valid type, correct filename format, and appropriate content describing codecov pipeline configuration fix.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch codecov

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/tests.yml:
- Around line 14-15: The reusable workflow secret parameter is case-sensitive:
change the secret key names under each job's secrets mapping from uppercase
CODECOV_TOKEN to the lowercase parameter expected by the reusable workflows
(e.g., replace secrets: CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} with
secrets: codecov_token: ${{ secrets.CODECOV_TOKEN }} for every job that uses the
reusable workflows such as the postgresql_18 job (uses:
./.github/workflows/single-postgres.yml) and the other 13 similar jobs so the
codecov_token parameter is correctly passed through.

ℹ️ Review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1060c2d and cc53804.

📒 Files selected for processing (2)
  • .github/workflows/tests.yml
  • newsfragments/+e9a4c914.misc.rst

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@fizyk fizyk merged commit 820fa69 into main Mar 1, 2026
52 checks passed
@fizyk fizyk deleted the codecov branch March 1, 2026 10:54
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.

1 participant