Skip to content

Docs: update paths from /cli to /cli-storybook in cli-storybook package.json #31799

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 4 commits into
base: next
Choose a base branch
from
Open

Docs: update paths from /cli to /cli-storybook in cli-storybook package.json #31799

wants to merge 4 commits into from

Conversation

ryomaejii
Copy link

@ryomaejii ryomaejii commented Jun 17, 2025 •
edited
Loading

issue: #31800

This PR fixes incorrect homepage and directory paths in package.json files for both CLI packages.

Changes

@storybook/cli package

  • Update homepage URL from code/lib/cli to code/lib/cli-storybook
  • Update repository directory from code/lib/cli to code/lib/cli-storybook

sb package (sub CLI)

  • Update homepage URL from code/lib/cli to code/lib/cli-sub
  • Update repository directory from code/lib/cli to code/lib/cli-sub

How to test

  • No functional changes, only metadata updates in package.json files

greptile-apps[bot]
greptile-apps bot reviewed Jun 17, 2025
View reviewed changes
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 reviewed, no comments
Edit PR Review Bot Settings | Greptile

@ryomaejii ryomaejii changed the title fix(cli): update package.json homepage and directory paths fix(cli): update paths from /cli to /cli-storybook in cli-storybook package.json Jun 17, 2025
@ryomaejii ryomaejii changed the title fix(cli): update paths from /cli to /cli-storybook in cli-storybook package.json fix(cli): update paths from /cli to /cli-storybook in cli-storybook package.json Jun 17, 2025
@ryomaejii ryomaejii changed the title fix(cli): update paths from /cli to /cli-storybook in cli-storybook package.json [Documentation]: update paths from /cli to /cli-storybook in cli-storybook package.json Jun 17, 2025
@ryomaejii ryomaejii changed the title [Documentation]: update paths from /cli to /cli-storybook in cli-storybook package.json Docs: update paths from /cli to /cli-storybook in cli-storybook package.json Jun 17, 2025
indigolain
indigolain reviewed Jun 18, 2025
View reviewed changes
Copy link
Member

@indigolain indigolain left a comment

Choose a reason for hiding this comment

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

Thank you for contributing!
I've noticed that some of the urls and paths may not be pointing to the intended ones.
Could you check them?

ryomaejii and others added 2 commits June 18, 2025 11:31
@ryomaejii ryomaejii requested a review from indigolain June 18, 2025 02:31
@nx-cloud Nx Cloud
Copy link

nx-cloud bot commented Jun 18, 2025

View your CI Pipeline Execution ↗ for commit d3c2e78.

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

☁️ Nx Cloud last updated this comment at 2025-06-18 02:38:59 UTC

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

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

@indigolain indigolain indigolain approved these changes

@Hardanish-Singh Hardanish-Singh Hardanish-Singh approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ryomaejii @indigolain @Hardanish-Singh