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

nuke "using npm programmatically" section #11130

Closed
wants to merge 1 commit into from
Closed

Conversation

ljharb
Copy link
Contributor

@ljharb ljharb commented Jan 13, 2016

The programmatic npm API is unsupported, and is not guaranteed not to break in non-major versions.

Removing this section so newcomers aren't encouraged to discover or use it.

The programmatic `npm` API is unsupported, and is not guaranteed not to break in non-major versions.

Removing this section so newcomers aren't encouraged to discover or use it.
@othiym23
Copy link
Contributor

This only really needs to be on master since it's mostly only seen on www.npmjs.com and GitHub, but we might as well remove it from both places because we don't support the API under npm@lts either.

@othiym23 othiym23 changed the title [readme] nuke "using npm programmatically" section nuke "using npm programmatically" section Jan 18, 2016
@iarna iarna added this to the next milestone Jan 20, 2016
iarna pushed a commit that referenced this pull request Jan 20, 2016
The programmatic `npm` API is unsupported, and is not guaranteed not to break in non-major versions.

Removing this section so newcomers aren't encouraged to discover or use it.

PR-URL: #11130
Credit: @ljharb
Reviewed-By: @othiym23
iarna pushed a commit that referenced this pull request Jan 21, 2016
The programmatic `npm` API is unsupported, and is not guaranteed not to break in non-major versions.

Removing this section so newcomers aren't encouraged to discover or use it.

PR-URL: #11130
Credit: @ljharb
Reviewed-By: @othiym23
zkat pushed a commit that referenced this pull request Jan 21, 2016
The programmatic `npm` API is unsupported, and is not guaranteed not to
break in non-major versions.

Removing this section so newcomers aren't encouraged to discover or use
it.

PR-URL: #11130
Credit: @ljharb
@iarna
Copy link
Contributor

iarna commented Jan 25, 2016

This was merged to 3.5.5 & 2.14.16! =)

@iarna iarna closed this Jan 25, 2016
@ljharb ljharb deleted the patch-1 branch January 25, 2016 23:07
@ljharb
Copy link
Contributor Author

ljharb commented Jan 25, 2016

Thanks!

@jescalan
Copy link

So just out of curiosity, if you need to use npm programmatically, what is the current recommendation? To run it through the command line in a child process?

@othiym23
Copy link
Contributor

@Jenius Yes. Eventually we'd like to have the CLI reduced to components that are independently usable to do all of the CLI's various tasks, but that's a very long-term project blocked on reworking how configuration is bootstrapped and managed within the current CLI implementation. Funnily enough, this is also part of what makes the current API unsuitable for third-party use.

@jescalan
Copy link

@othiym23 Totally understand how that would be a pretty large challenge. I'm sure whenever that does happen, the organization of the API will be much better, looking forward to it! Child process it is for now then. Thanks for the quick response 🙌

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.

4 participants