Skip to content
This repository has been archived by the owner on Aug 11, 2022. It is now read-only.

Allow npm version to be run from subdirectories #10627

Closed
wants to merge 1 commit into from

Conversation

ekmartin
Copy link
Contributor

Fixes #9099 by using the local package.json path when running git add. Was this something like what you were thinking of @othiym23? If it was I can add a small description of the behavior to the docs, just wanted to check first.

@othiym23
Copy link
Contributor

othiym23 commented Dec 4, 2015

I haven't had time to review this thoroughly, but if you think a note in the docs would be helpful, by all means add it! We'll try to get this into next week's releases.

@ekmartin
Copy link
Contributor Author

ekmartin commented Dec 6, 2015

No problem! Not sure if an update to the docs is needed, but I'm not familiar with the differences of opinion you were mentioning here. If there's other results that would make sense when you run version from a subdirectory I agree that it probably should be clarified.

othiym23 pushed a commit that referenced this pull request Dec 11, 2015
@othiym23
Copy link
Contributor

Landed for npm@3 as 954fa67, and will do the same for npm@2 shortly. This is a well-done patch! Thanks for taking the time to put it together, and for including a test!

othiym23 pushed a commit that referenced this pull request Dec 11, 2015
@othiym23
Copy link
Contributor

Cherry-picked for npm@2 as 529fa1f. Thanks again!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants