Skip to content

Conversation

@9aoy
Copy link
Contributor

@9aoy 9aoy commented Sep 11, 2025

Summary

support clean coverage directory before running tests.

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings September 11, 2025 07:31
@netlify
Copy link

netlify bot commented Sep 11, 2025

Deploy Preview for rstest-dev ready!

Name Link
🔨 Latest commit 27c3810
🔍 Latest deploy log https://app.netlify.com/projects/rstest-dev/deploys/68c27d3d5875b900083405b8
😎 Deploy Preview https://deploy-preview-558--rstest-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for a clean option to the coverage configuration that automatically cleans the coverage directory before running tests.

  • Added a new clean boolean option to CoverageOptions with a default value of true
  • Updated type definitions to properly handle the coverage configuration in project configs
  • Integrated coverage cleanup functionality through a new core plugin

Reviewed Changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
packages/core/src/types/coverage.ts Added clean boolean option to CoverageOptions
packages/core/src/types/config.ts Updated type definitions to handle normalized coverage options
packages/core/src/config.ts Set default value for clean option to true
packages/core/src/core/rsbuild.ts Added coverage core plugin and command condition
website/docs/*/config/test/projects.mdx Updated documentation to exclude coverage from ProjectConfig
e2e/test-coverage/index.test.ts Updated test to verify coverage directory cleaning

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@9aoy 9aoy merged commit 8f6c612 into main Sep 11, 2025
16 checks passed
@9aoy 9aoy deleted the coverage-clean branch September 11, 2025 07:52
@9aoy 9aoy mentioned this pull request Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant