Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

valentinpalkovic
Copy link
Contributor

@valentinpalkovic valentinpalkovic commented Apr 1, 2025

Closes #

What I did

Additionally update related configurations and documentation. Introduce new @storybook/addon-vitest with necessary files and structure for testing integration in Storybook.

Checklist for Contributors

Testing

The changes in this PR are covered in the following automated tests:

  • stories
  • unit tests
  • integration tests
  • end-to-end tests

Manual testing

This section is mandatory for all contributions. If you believe no manual test is necessary, please state so explicitly. Thanks!

Documentation

  • Add or update documentation reflecting your changes
  • If you are deprecating/removing a feature, make sure to update
    MIGRATION.MD

Checklist for Maintainers

  • When this PR is ready for testing, make sure to add ci:normal, ci:merged or ci:daily GH label to it to run a specific set of sandboxes. The particular set of sandboxes can be found in code/lib/cli-storybook/src/sandbox-templates.ts

  • Make sure this PR contains one of the labels below:

    Available labels
    • bug: Internal changes that fixes incorrect behavior.
    • maintenance: User-facing maintenance tasks.
    • dependencies: Upgrading (sometimes downgrading) dependencies.
    • build: Internal-facing build tooling & test updates. Will not show up in release changelog.
    • cleanup: Minor cleanup style change. Will not show up in release changelog.
    • documentation: Documentation only changes. Will not show up in release changelog.
    • feature request: Introducing a new feature.
    • BREAKING CHANGE: Changes that break compatibility in some way with current major version.
    • other: Changes that don't fit in the above categories.

🦋 Canary release

This pull request has been released as version 0.0.0-pr-31014-sha-9708fd9e. Try it out in a new sandbox by running npx [email protected] sandbox or in an existing project with npx [email protected] upgrade.

More information
Published version 0.0.0-pr-31014-sha-9708fd9e
Triggered by @valentinpalkovic
Repository storybookjs/storybook
Branch valentin/rename-addon-test-to-addon-vitest
Commit 9708fd9e
Datetime Fri Apr 4 09:24:20 UTC 2025 (1743758660)
Workflow run 14262246549

To request a new release of this pull request, mention the @storybookjs/core team.

core team members can create a new canary release here or locally with gh workflow run --repo storybookjs/storybook canary-release-pr.yml --field pr=31014

Greptile Summary

This PR renames the experimental test addon to addon-vitest and revises all related configuration, documentation, and internal references for seamless integration in Storybook.

  • Updated core configuration files (code/.storybook/main.ts, preview.tsx, vitest.config.ts) to import and use '@storybook/addon-vitest'.
  • Renamed addon files in code/addons/vitest (e.g. README.md, manager.js, package.json, postinstall scripts) to align with the new naming.
  • Updated migration documents (MIGRATION.md) and documentation snippets (docs/_snippets/*) to reflect addon-vitest.
  • Revised internal references in core/version mappings and automigrate scripts to ensure consistency with the new addon name.

Additionally update related configurations and documentation. Introduce new `@storybook/addon-vitest` with necessary files and structure for testing integration in Storybook.
Copy link
Contributor

github-actions bot commented Apr 1, 2025

Fails
🚫 PR is marked with "BREAKING CHANGE" label.

Generated by 🚫 dangerJS against 9708fd9

Copy link

nx-cloud bot commented Apr 1, 2025

View your CI Pipeline Execution ↗ for commit 9708fd9.

Command Status Duration Result
nx run-many -t build --parallel=3 ✅ Succeeded 1m 34s View ↗

☁️ Nx Cloud last updated this comment at 2025-04-04 08:55:08 UTC

…st" for consistency across component and e2e tests.
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

LGTM

87 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

Copy link
Contributor

@jonniebigodes jonniebigodes left a comment

Choose a reason for hiding this comment

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

@valentinpalkovic, thanks for taking the time to put this together and update the documentation as well. I appreciate it 🙏 ! I left some items for you to look into when you can. I'm approving this to unblock you.

… corrections. Rename `@storybook/experimental-addon-test` to `@storybook/addon-vitest` in the documentation and ensure consistent formatting for links and section titles.
…test-panel

Addon Test: Update panel title references from "Component tests"
@valentinpalkovic valentinpalkovic merged commit 86376ed into next Apr 4, 2025
41 of 54 checks passed
@valentinpalkovic valentinpalkovic deleted the valentin/rename-addon-test-to-addon-vitest branch April 4, 2025 09:52
@github-actions github-actions bot mentioned this pull request Apr 4, 2025
9 tasks
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.

2 participants