-
Notifications
You must be signed in to change notification settings - Fork 38
epic: starship v2 initialization #698
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.
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
Conversation
v2: types defination for the starship config
There was a problem hiding this 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 updates version references and adjusts file paths for the new v2 monorepo layout.
- Bumps package versions to 3.14.1 in lerna.json and changelogs
- Redirects README badges and links from
clients/jsto the newpackagesdirectory - Updates GitHub Actions workflows to run against the
packagesfolder
Reviewed Changes
Copilot reviewed 98 out of 98 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/packages/cli/CHANGELOG.md | Added v3.14.1 and v3.14.0 entries; version jump noted |
| packages/lerna.json | Bumped monorepo version to 3.14.1 and allowed v2 branch |
| packages/README.md | Updated badge links and package directory URLs |
| packages/CHANGELOG.md | Added multiple new version entries up to v3.14.1 |
| .github/workflows/run-client-tests.yml | Changed paths and working-directory to packages |
| .github/workflows/pr-tests.yaml | Updated install steps to packages directory |
| .github/workflows/lint-client.yml | Updated paths and working-directory to packages |
Comments suppressed due to low confidence (4)
packages/README.md:11
- The license link points to
packages/LICENSE, but the repository's LICENSE file lives at the root. Please update this URL to the correct path (e.g.,/LICENSE).
</a><a href="https://github.com/hyperweb-io/starship/blob/main/packages/LICENSE"><img height="20" src="https://img.shields.io/badge/license-MIT-blue.svg"></a>
packages/README.md:28
- The link target includes a duplicated
packagessegment (packages/packages/...). It should point directly to the package folder (e.g.,packages/starshipjs), so the link works correctly.
[`StarshipJS`](https://github.com/hyperweb-io/starship/tree/main/packages/packages/starshipjs): A JavaScript library providing...
.github/workflows/run-client-tests.yml:25
- Setting
working-directorytopackagesruns commands in the monorepo root packages folder, not in a specific package. Specify the actual CLI package path (e.g.,packages/cli) to ensure the intended tests are executed.
working-directory: packages
packages/packages/cli/CHANGELOG.md:10
- The changelog for the CLI package jumps from v3.12.0 to v3.14.0, omitting v3.13.0. Please add an entry for v3.13.0 or note the intentional version skip to maintain clarity in the release history.
# [3.14.0](https://github.com/hyperweb-io/starship/compare/v3.13.0...v3.14.0) (2025-06-11)
Part of: #695