Skip to content

chore: group vitest dependabot updates#18

Merged
mohanagy merged 1 commit intomainfrom
chore/dependency-policy
Apr 27, 2026
Merged

chore: group vitest dependabot updates#18
mohanagy merged 1 commit intomainfrom
chore/dependency-policy

Conversation

@mohanagy
Copy link
Copy Markdown
Owner

@mohanagy mohanagy commented Apr 27, 2026

Summary

  • group vitest and @vitest/coverage-v8 into the same Dependabot npm update batch
  • extend package-metadata tests to enforce the grouping rule alongside version alignment

Test Plan

  • npm run test:run -- tests/unit/package-metadata.test.ts
  • npx -y node@20.20.2 ./node_modules/typescript/bin/tsc --noEmit
  • npx -y node@20.20.2 ./node_modules/vitest/vitest.mjs run

Summary by CodeRabbit

  • Chores

    • Updated Dependabot configuration to group test-related dependency updates into a single batch.
  • Tests

    • Added validation for Dependabot grouping configuration.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 27, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The changes add Dependabot grouping configuration to batch vitest and @vitest/coverage-v8 dependency updates into a single group, and introduce a unit test to validate this grouping configuration is properly set.

Changes

Cohort / File(s) Summary
Dependabot Configuration
.github/dependabot.yml
Adds a groups rule under the npm update entry to group vitest and @vitest/coverage-v8 dependency updates into a single named group called test-tooling.
Test Validation
tests/unit/package-metadata.test.ts
Adds a helper function to read dependabot.yml and a new unit test that validates the presence of grouping configuration, the test-tooling group, patterns, and expected pattern strings for both test dependencies.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 Hop, hop, the test tools align,
In groups they now bundle, so fine!
Vitest and coverage, together they stay,
And tests make sure they don't stray away! 🧪✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: grouping vitest and related dependencies in Dependabot configuration.
Description check ✅ Passed The description covers the summary and testing sections, but omits the checklist and related issues sections from the template.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/dependency-policy

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.

@mohanagy mohanagy merged commit fb011a9 into main Apr 27, 2026
6 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.

1 participant