Skip to content

Conversation

@XhmikosR
Copy link
Contributor

Needs tests or ESLint so I'll rebase it after #30 :)

@XhmikosR XhmikosR marked this pull request as ready for review March 20, 2020 14:24
@XhmikosR
Copy link
Contributor Author

@sagarkarira the package-lock.json was not right in master either. The last patch I have here fixes it. Preview: https://github.com/XhmikosR/coronavirus-tracker-cli/runs/522111636

Note that this will run from the next PR for security reasons.

@sagarkarira
Copy link
Owner

sagarkarira commented Mar 20, 2020 via email

@XhmikosR
Copy link
Contributor Author

This fixes it for sure as you can see, since npm ci would error out otherwise :)

@sagarkarira
Copy link
Owner

Great, I am merging this.

@sagarkarira sagarkarira merged commit bc6ad2a into sagarkarira:master Mar 20, 2020
@XhmikosR XhmikosR deleted the ci branch March 20, 2020 14:38
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