Skip to content
This repository has been archived by the owner on Sep 21, 2023. It is now read-only.

Add NPM_CONFIG_REGISTRY to environment docs #483

Merged
merged 3 commits into from
Dec 10, 2022

Conversation

NickTomlin
Copy link
Contributor

Related to: denoland/deno#16105

I think there is probably more work to do to flesh out support for private registries (e.g would be great to formalize things like auth configuration) but this at least documents a way to start integrating with a private registry.

Related to: denoland/deno#16105

I think there is probably more work to do to flesh out support for private registries (e.g would be great to formalize things like [auth configuration](https://docs.npmjs.com/cli/v9/configuring-npm/npmrc#auth-related-configuration)) but this at least documents a way to start integrating with a private registry.
@CLAassistant
Copy link

CLAassistant commented Dec 5, 2022

CLA assistant check
All committers have signed the CLA.

@bartlomieju
Copy link
Member

@dsherret please take a look

@dsherret
Copy link
Member

dsherret commented Dec 5, 2022

This is purposefully not documented because it hasn't been properly thought out or discussed yet. I don't think we should merge this at this time.

@NickTomlin NickTomlin closed this Dec 5, 2022
@dsherret dsherret changed the title Add DENO_NPM_REGISTRY to environment docs Add NPM_CONFIG_REGISTRY to environment docs Dec 7, 2022
@dsherret dsherret reopened this Dec 7, 2022
Copy link
Member

@dsherret dsherret left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for NPM_CONFIG_REGISTRY. Thanks @NickTomlin!

@bartlomieju bartlomieju merged commit bc93dfa into denoland:main Dec 10, 2022
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.

4 participants