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

Remove lock file from repo and update deps #242

Merged
merged 4 commits into from
Feb 1, 2020
Merged

Conversation

fanatid
Copy link
Collaborator

@fanatid fanatid commented Jan 29, 2020

I was wrong about pushing lock for to repo (#220, #226).

  • Yarn@2 changed lock file format. Current lock file need to be updated. But people can use npm instead yarn.
  • Lock file for end projects, not libraries.

While lock file was not included in published package it was able add some confusion.

Also applied sort-package-json for package.json files.

@fanatid fanatid merged commit 00d5851 into master Feb 1, 2020
@fanatid fanatid deleted the lock-file-remove branch February 1, 2020 11:30
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.

1 participant