Skip to content

docs: clarify --min-coverage and configuration - #1685

Merged
marcossevilla merged 2 commits into
mainfrom
vgv-ai-bot/issue-1684
Aug 11, 2026
Merged

marcossevilla merged 2 commits into
mainfrom
vgv-ai-bot/issue-1684

Conversation

@unicoderbot

@unicoderbot unicoderbot Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Updates the help text for the --min-coverage flag to document that it implicitly enables coverage collection, matching the existing documentation for --show-uncovered.

Changes

  • Updated --min-coverage help text in lib/src/commands/test/test.dart
  • Updated --min-coverage help text in lib/src/commands/dart/commands/dart_test_command.dart
  • Updated documentation in site/docs/commands/test.md
  • Updated corresponding test expectations

The help text now reads: "Whether to enforce a minimum coverage percentage. Implicitly enables coverage collection when used alone." to match the behavior of --show-uncovered.

Closes

#1684

Closes #1684

Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@marcossevilla

Copy link
Copy Markdown
Member

@vgvbot investigate and fix CI failures

@unicoderbot

unicoderbot Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

On it, I'll address the review comments and push the changes.

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Reviewing feedback
  • Working on changes
  • Pushing updates

I've pushed changes to address the feedback.

Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@marcossevilla marcossevilla changed the title docs: clarify --min-coverage implicitly enables coverage docs: clarify --min-coverage and configuration Aug 11, 2026
@marcossevilla
marcossevilla merged commit b5c2457 into main Aug 11, 2026
25 checks passed
@marcossevilla
marcossevilla deleted the vgv-ai-bot/issue-1684 branch August 11, 2026 16:42
@vgvbot vgvbot mentioned this pull request Aug 11, 2026
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