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

Release v2.3.1 #84

Merged
merged 3 commits into from
Jun 26, 2022
Merged

Release v2.3.1 #84

merged 3 commits into from
Jun 26, 2022

Conversation

tejzpr
Copy link
Collaborator

@tejzpr tejzpr commented Jun 20, 2022

Updated package.json version to reflect license change.

Added GitHub action to create npm package. This action creates a new NPM package when someone pushes a commit with format "Release vx.x.x". Depends on npm token which has to be defined in GitHub repo secrets with key name "NPM_AUTH_TOKEN". @nelsonic Please add this GitHub repo secret before merge

@tejzpr tejzpr changed the title Action to publish npm package GitHub Action to publish npm package Jun 20, 2022
Update package.json version for npm + license change
@tejzpr tejzpr changed the title GitHub Action to publish npm package Release v2.3.1 Jun 20, 2022
@tejzpr tejzpr requested a review from nelsonic June 20, 2022 14:51
@nelsonic nelsonic self-assigned this Jun 20, 2022
@nelsonic
Copy link
Member

FYI: AFK without OTP app RN so can't add the NPM_AUTH_TOKEN for auto-publish. ⏳
Will do this once I'm back at my desk this evening. 👨‍💻

@tejzpr
Copy link
Collaborator Author

tejzpr commented Jun 26, 2022

@nelsonic please merge.

@nelsonic nelsonic mentioned this pull request Jun 26, 2022
Copy link
Member

@nelsonic nelsonic left a comment

Choose a reason for hiding this comment

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

Thanks @tejzpr :shipit:

@nelsonic nelsonic merged commit e4781fe into dwyl:master Jun 26, 2022
@nelsonic
Copy link
Member

hapi-error@2.3.1 published to NPM (manually) until I can get the auto-deploy working. ⏳
Thanks again @tejzpr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants