Skip to content

Conversation

@tomdavies73
Copy link
Contributor

Proposed behaviour

Excludes .storybook and scripts directories from the coverage report

Screenshot 2026-02-10 at 15 20 17

Current behaviour

Currently, .storybook and scripts directories are included in the coverage report

Screenshot 2026-02-10 at 15 41 24

Checklist

  • Commits follow our style guide
  • Related issues linked in commit messages if required
  • Screenshots are included in the PR if useful
  • All themes are supported if required
  • Unit tests added or updated if required
  • Playwright automation tests added or updated if required
  • Storybook added or updated if required
  • Translations added or updated (including creating or amending translation keys table in storybook) if required
  • Typescript d.ts file added or updated if required
  • Related docs have been updated if required

QA

  • Tested in provided StackBlitz sandbox/Storybook
  • Add new Playwright test coverage if required
  • Carbon implementation matches Design System/designs
  • UI Tests GitHub check reviewed if required

Additional context

Testing instructions

@tomdavies73 tomdavies73 marked this pull request as ready for review February 11, 2026 08:50
@tomdavies73 tomdavies73 requested a review from a team as a code owner February 11, 2026 08:50
@tomdavies73 tomdavies73 merged commit ce2c48b into master Feb 11, 2026
29 checks passed
@tomdavies73 tomdavies73 deleted the add_coverage_ignore_paths branch February 11, 2026 08:58
@carbonci
Copy link
Collaborator

🎉 This PR is included in version 158.28.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants