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

Move travis ci & semantic release steps to github actions #27

Merged
merged 1 commit into from
Nov 19, 2020

Conversation

georgebabey
Copy link
Contributor

No description provided.

@georgebabey georgebabey force-pushed the gbabey/travis-to-github-actions branch from 4395bf3 to 9c2c863 Compare November 19, 2020 13:55
with:
node-version: 12
- name: Install dependencies
run: npm ci
Copy link
Contributor

Choose a reason for hiding this comment

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

I like that we use npm ci instead of npm i !!!

Copy link
Contributor

@binodpant binodpant left a comment

Choose a reason for hiding this comment

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

left minor comments, but nothing I see to not merge this

@georgebabey georgebabey force-pushed the gbabey/travis-to-github-actions branch from 9c2c863 to b36e07a Compare November 19, 2020 14:37
@georgebabey georgebabey force-pushed the gbabey/travis-to-github-actions branch from b36e07a to 485d325 Compare November 19, 2020 14:57
@georgebabey georgebabey merged commit ec22ab4 into master Nov 19, 2020
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.

None yet

2 participants