Skip to content
This repository has been archived by the owner on Jul 30, 2018. It is now read-only.

Do not ignore package-lock.json #711

Merged
merged 1 commit into from
Oct 10, 2017

Conversation

agubler
Copy link
Member

@agubler agubler commented Oct 9, 2017

Type: feature

The following has been addressed in the PR:

  • There is a related issue
  • All code matches the style guide
  • Unit or Functional tests are included in the PR

Description:

We shouldn't need to ignore the package-lock.json anymore, it causes a failure in the grunt release task as it is using npm version which implicitly tries to add the file to commit (and the release fails). As the peer dependencies are installed with a pre publish script we should always get the latest version according to the semver dependency version.

@kitsonk
Copy link
Member

kitsonk commented Oct 9, 2017

Probably should also open an issue on meta for this so we can roll it out everywhere.

@dylans dylans added this to the 2017.10 milestone Oct 10, 2017
@agubler agubler merged commit a068180 into dojo:master Oct 10, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants