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(deps): remove 'npm' and 'i' dependencies #387

Merged
merged 1 commit into from
May 31, 2021

Conversation

oscard0m
Copy link
Member

Fix #386

@oscard0m oscard0m requested a review from gr2m May 31, 2021 12:23
@wolfy1339
Copy link
Member

npm@6 is currently used to generate the lockfile.

Could you revert the changes to the lock file and use npm@6?

@oscard0m
Copy link
Member Author

npm@6 is currently used to generate the lockfile.

Could you revert the changes to the lock file and use npm@6?

Good catch @wolfy1339

@oscard0m oscard0m requested a review from wolfy1339 May 31, 2021 12:32
@oscard0m oscard0m force-pushed the remove-deps-added-by-mistake branch from 8b87d17 to ce4b9a7 Compare May 31, 2021 12:37
@oscard0m oscard0m merged commit 490f338 into master May 31, 2021
@oscard0m oscard0m deleted the remove-deps-added-by-mistake branch May 31, 2021 13:14
@github-actions
Copy link
Contributor

github-actions bot commented Jun 2, 2021

🎉 This PR is included in version 10.4.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

npm added as a production dependency - typo?
2 participants