Skip to content
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

chore: add changesets #94

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

chore: add changesets #94

wants to merge 1 commit into from

Conversation

bennypowers
Copy link
Member

should work

Add NPM_TOKEN before merging (certainly, before releasing)

@netlify
Copy link

netlify bot commented Jul 15, 2021

✔️ Deploy Preview for custom-elements-manifest-analyzer ready!

🔨 Explore the source changes: f7b9cc9

🔍 Inspect the deploy log: https://app.netlify.com/sites/custom-elements-manifest-analyzer/deploys/6101022264904300079f88ad

😎 Browse the preview: https://deploy-preview-94--custom-elements-manifest-analyzer.netlify.app

@thepassle
Copy link
Member

Sweet

Where do I get the NPM token from, and where do I add it? 🙈

do I need to add the github token as well? or it already knows?

@bennypowers
Copy link
Member Author

github token should be provided automatically by actions, so I don't think you need to do anything specific. You can generate a new token at npmjs.com, in the profile dropdown. if you have 2fa select 'automation token' and set it as NPM_TOKEN in the repo settings under secrets

@daKmoR
Copy link
Member

daKmoR commented Jul 24, 2021

you can login with modern web into npm and create a new automation token

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.

3 participants