-
Notifications
You must be signed in to change notification settings - Fork 5
adding changeset #208
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
base: main
Are you sure you want to change the base?
adding changeset #208
Conversation
| ```bash | ||
| changeset | ||
| ``` |
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.
A command changeset is not added to my PATH by npm install. Is there some setup missing from this README?
Note that Changesets documents npx @changesets/cli instead of a command changeset.
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.
i've adjusted the package.lock file to include all of these changeset commands referenced in the README and adjusted the README to include npm run commands instead.
Can you confirm that this works as documented now @WULCAN ?
| "lyra" | ||
| ], | ||
| "devDependencies": { | ||
| "@changesets/cli": "^2.29.4", |
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.
Our package-lock is not fully updated after this change. Let's add the changes npm install does to our package-lock.
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.
resolved via 8bc4959
webapp/CHANGELOG.md
Outdated
| ## 0.4.1 | ||
|
|
||
| ### Patch Changes | ||
|
|
||
| - Added changesets | ||
|
|
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.
0.4.1 is already tagged, it's too late to add changesets to 0.4.1 now.
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.
now even more time has passed and it's gonna be 0.7.1
webapp/package.json
Outdated
| { | ||
| "name": "lyra-webapp", | ||
| "version": "0.4.0", | ||
| "version": "0.4.1", |
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.
Our package-lock is not updated after this change. Please add the changes npm install does to our package-lock.
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.
resolved via 8bc4959
Signed-off-by: Alexander Schreiner <[email protected]>
bb8acd1 to
8bc4959
Compare
Description
This PR introduced changesets
Changes
[Add a list of features added/changed, bugs fixed etc]
Related issues
Resolves #200