Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

CLI: Fix qwik init #29632

Merged
merged 1 commit into from
Nov 17, 2024
Merged

CLI: Fix qwik init #29632

merged 1 commit into from
Nov 17, 2024

Conversation

shilman
Copy link
Member

@shilman shilman commented Nov 17, 2024

Closes N/A

What I did

See https://github.com/literalpie/storybook-framework-qwik/blob/main/CHANGELOG.md#040

🦋 Canary release

This PR does not have a canary release associated. You can request a canary release of this pull request by mentioning the @storybookjs/core team here.

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

Greptile Summary

This PR standardizes the testing library usage across all Storybook frameworks by removing special handling for Qwik and using @storybook/test consistently.

  • Modified code/lib/create-storybook/src/generators/baseGenerator.ts to remove Qwik-specific testing library condition
  • Aligns with Qwik framework's migration to @storybook/test in v0.4.0
  • Simplifies testing setup by using a single consistent package (@storybook/test) across all frameworks
  • Removes technical debt from framework-specific testing library variations

@shilman shilman added bug compatibility with other tools cli patch:yes Bugfix & documentation PR that need to be picked to main branch ci:normal labels Nov 17, 2024
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

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

Copy link

nx-cloud bot commented Nov 17, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 85d1d2a. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 2 targets

Sent with 💌 from NxCloud.

@shilman shilman merged commit 6f0f798 into next Nov 17, 2024
24 of 31 checks passed
@shilman shilman deleted the shilman/fix-qwik-init branch November 17, 2024 06:17
yannbf pushed a commit that referenced this pull request Nov 20, 2024
CLI: Fix qwik init
(cherry picked from commit 6f0f798)
@github-actions github-actions bot added the patch:done Patch/release PRs already cherry-picked to main/release branch label Nov 20, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug ci:normal cli compatibility with other tools patch:done Patch/release PRs already cherry-picked to main/release branch patch:yes Bugfix & documentation PR that need to be picked to main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant