Skip to content

fix: use npm for release#2339

Merged
millotp merged 1 commit intomainfrom
fix/release
Nov 5, 2024
Merged

fix: use npm for release#2339
millotp merged 1 commit intomainfrom
fix/release

Conversation

@millotp
Copy link
Copy Markdown
Contributor

@millotp millotp commented Nov 4, 2024

yarn publish doesn't exists anymore

@millotp millotp self-assigned this Nov 4, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Nov 4, 2024

Deploy Preview for docsearch canceled.

Name Link
🔨 Latest commit 1e409c1
🔍 Latest deploy log https://app.netlify.com/sites/docsearch/deploys/67291de76ca03000088e2239

},
publishCommand({ tag }) {
return `yarn publish --access public --tag ${tag}`;
return `npm publish --tag ${tag}`;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Isn't there something to put for the token to use it on the ci?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

it should automatically pick up the NPM_AUTH_TOKEN that's on the CI

@millotp millotp merged commit 32ec976 into main Nov 5, 2024
@millotp millotp deleted the fix/release branch November 5, 2024 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants