Skip to content

ci: add coverage reporting and CI/coverage badges to README (#42)#46

Merged
matisiekpl merged 1 commit into
matisiekpl:mainfrom
chrisdebian:fix/42-ci-coverage-badges
Jun 10, 2026
Merged

ci: add coverage reporting and CI/coverage badges to README (#42)#46
matisiekpl merged 1 commit into
matisiekpl:mainfrom
chrisdebian:fix/42-ci-coverage-badges

Conversation

@chrisdebian

Copy link
Copy Markdown
Contributor

Summary

Fixes #42.

  • Adds --coverage flag to flutter test so coverage/lcov.info is generated on every CI run.
  • Uploads the report to Codecov via codecov/codecov-action@v5. fail_ci_if_error: false means CI continues to pass even before the CODECOV_TOKEN secret is configured.
  • Adds CI status and Codecov badges to README, below the screenshot.
  • Bumps actions/checkout v3 → v4 and keeps subosito/flutter-action@v2.

To activate the coverage badge: add CODECOV_TOKEN as a repository secret at Settings → Secrets and variables → Actions, using the token from codecov.io/gh/matisiekpl/aed_map.

Test plan

  • CI workflow syntax valid
  • flutter test --coverage runs locally and produces coverage/lcov.info

Closes #42

…pl#42)

- flutter test --coverage generates coverage/lcov.info
- codecov-action uploads report; fail_ci_if_error: false so CI still
  passes if the Codecov token is not yet configured
- actions/checkout and subosito/flutter-action bumped to latest
- CI status and Codecov badges added to README

Note: coverage badge will show once CODECOV_TOKEN is added as a
repository secret at Settings → Secrets → Actions.
@netlify

netlify Bot commented Jun 4, 2026

Copy link
Copy Markdown

👷 Deploy request for aedmap pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 3be6320

@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@matisiekpl matisiekpl merged commit 007942e into matisiekpl:main Jun 10, 2026
2 checks passed
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.

Add CI status and code coverage badges to README

3 participants