docs: standardize env vars to uppercase convention - #8756

Merged
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case
Nov 25, 2025
Merged

docs: standardize env vars to uppercase convention#8756
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case

Conversation

@MaxBlack-dev

Copy link
Copy Markdown
Contributor

Description

Standardizes all environment variable references in documentation to use uppercase naming convention (NPM_CONFIG_) instead of lowercase (npm_config_).

Changes

  • Changed npm_config_binroot to NPM_CONFIG_BINROOT in scripts.md
  • Changed npm_config_prefix to NPM_CONFIG_PREFIX in npm.md (2 instances)
  • Changed npm_config_ to NPM_CONFIG_ in npm.md
  • Changed npm_config_key to NPM_CONFIG_KEY in npm.md

Fixes

Closes#4891

Rationale

The documentation had inconsistent casing for environment variables - some references used uppercase (NPM_CONFIG_USERCONFIG) while others used lowercase (npm_config_prefix). Following the standard convention, environment variables should always be referenced in uppercase. While npm internally converts both to lowercase, using uppercase in documentation follows the de-facto standard and avoids confusion.

Type of Change

Documentation update

@MaxBlack-dev
MaxBlack-dev requested a review from a team as a code ownerNovember 19, 2025 23:19
@owlstronaut
owlstronaut merged commit 17e154c into npm:latestNov 25, 2025
8 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Nov 25, 2025
@ryenus

ryenus commented Nov 26, 2025

Copy link
Copy Markdown

Wow, finally, thank you @MaxBlack-dev so much for this! 🎉

BTW, I vaguely remember to have seen someone mentioned that when both the lowercase and uppercase environment variables exist, like having both npm_config_userconfig and NPM_CONFIG_USERCONFIG set, then one of the two would take precedence due to the way things have been implemented. I might be wrong though, I wonder if there's any plan to clarify that part as well?

Sign up for freeto 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.

[DOCS] npm_config_userconfig vs NPM_CONFIG_USERCONFIG when referred as environment variable

3 participants

@MaxBlack-dev@ryenus@owlstronaut
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

docs: standardize env vars to uppercase convention - #8756

Merged
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case
Nov 25, 2025
Merged

docs: standardize env vars to uppercase convention#8756
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case

Conversation

@MaxBlack-dev

Copy link
Copy Markdown
Contributor

Description

Standardizes all environment variable references in documentation to use uppercase naming convention (NPM_CONFIG_) instead of lowercase (npm_config_).

Changes

  • Changed npm_config_binroot to NPM_CONFIG_BINROOT in scripts.md
  • Changed npm_config_prefix to NPM_CONFIG_PREFIX in npm.md (2 instances)
  • Changed npm_config_ to NPM_CONFIG_ in npm.md
  • Changed npm_config_key to NPM_CONFIG_KEY in npm.md

Fixes

Closes#4891

Rationale

The documentation had inconsistent casing for environment variables - some references used uppercase (NPM_CONFIG_USERCONFIG) while others used lowercase (npm_config_prefix). Following the standard convention, environment variables should always be referenced in uppercase. While npm internally converts both to lowercase, using uppercase in documentation follows the de-facto standard and avoids confusion.

Type of Change

Documentation update

@MaxBlack-dev
MaxBlack-dev requested a review from a team as a code ownerNovember 19, 2025 23:19
@owlstronaut
owlstronaut merged commit 17e154c into npm:latestNov 25, 2025
8 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Nov 25, 2025
@ryenus

ryenus commented Nov 26, 2025

Copy link
Copy Markdown

Wow, finally, thank you @MaxBlack-dev so much for this! 🎉

BTW, I vaguely remember to have seen someone mentioned that when both the lowercase and uppercase environment variables exist, like having both npm_config_userconfig and NPM_CONFIG_USERCONFIG set, then one of the two would take precedence due to the way things have been implemented. I might be wrong though, I wonder if there's any plan to clarify that part as well?

Sign up for freeto 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.

[DOCS] npm_config_userconfig vs NPM_CONFIG_USERCONFIG when referred as environment variable

3 participants

@MaxBlack-dev@ryenus@owlstronaut
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

docs: standardize env vars to uppercase convention - #8756

Merged
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case
Nov 25, 2025
Merged

docs: standardize env vars to uppercase convention#8756
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case

Conversation

@MaxBlack-dev

Copy link
Copy Markdown
Contributor

Description

Standardizes all environment variable references in documentation to use uppercase naming convention (NPM_CONFIG_) instead of lowercase (npm_config_).

Changes

  • Changed npm_config_binroot to NPM_CONFIG_BINROOT in scripts.md
  • Changed npm_config_prefix to NPM_CONFIG_PREFIX in npm.md (2 instances)
  • Changed npm_config_ to NPM_CONFIG_ in npm.md
  • Changed npm_config_key to NPM_CONFIG_KEY in npm.md

Fixes

Closes#4891

Rationale

The documentation had inconsistent casing for environment variables - some references used uppercase (NPM_CONFIG_USERCONFIG) while others used lowercase (npm_config_prefix). Following the standard convention, environment variables should always be referenced in uppercase. While npm internally converts both to lowercase, using uppercase in documentation follows the de-facto standard and avoids confusion.

Type of Change

Documentation update

@MaxBlack-dev
MaxBlack-dev requested a review from a team as a code ownerNovember 19, 2025 23:19
@owlstronaut
owlstronaut merged commit 17e154c into npm:latestNov 25, 2025
8 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Nov 25, 2025
@ryenus

ryenus commented Nov 26, 2025

Copy link
Copy Markdown

Wow, finally, thank you @MaxBlack-dev so much for this! 🎉

BTW, I vaguely remember to have seen someone mentioned that when both the lowercase and uppercase environment variables exist, like having both npm_config_userconfig and NPM_CONFIG_USERCONFIG set, then one of the two would take precedence due to the way things have been implemented. I might be wrong though, I wonder if there's any plan to clarify that part as well?

Sign up for freeto 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.

[DOCS] npm_config_userconfig vs NPM_CONFIG_USERCONFIG when referred as environment variable

3 participants

@MaxBlack-dev@ryenus@owlstronaut
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

docs: standardize env vars to uppercase convention - #8756

Merged
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case
Nov 25, 2025
Merged

docs: standardize env vars to uppercase convention#8756
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case

Conversation

@MaxBlack-dev

Copy link
Copy Markdown
Contributor

Description

Standardizes all environment variable references in documentation to use uppercase naming convention (NPM_CONFIG_) instead of lowercase (npm_config_).

Changes

  • Changed npm_config_binroot to NPM_CONFIG_BINROOT in scripts.md
  • Changed npm_config_prefix to NPM_CONFIG_PREFIX in npm.md (2 instances)
  • Changed npm_config_ to NPM_CONFIG_ in npm.md
  • Changed npm_config_key to NPM_CONFIG_KEY in npm.md

Fixes

Closes#4891

Rationale

The documentation had inconsistent casing for environment variables - some references used uppercase (NPM_CONFIG_USERCONFIG) while others used lowercase (npm_config_prefix). Following the standard convention, environment variables should always be referenced in uppercase. While npm internally converts both to lowercase, using uppercase in documentation follows the de-facto standard and avoids confusion.

Type of Change

Documentation update

@MaxBlack-dev
MaxBlack-dev requested a review from a team as a code ownerNovember 19, 2025 23:19
@owlstronaut
owlstronaut merged commit 17e154c into npm:latestNov 25, 2025
8 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Nov 25, 2025
@ryenus

ryenus commented Nov 26, 2025

Copy link
Copy Markdown

Wow, finally, thank you @MaxBlack-dev so much for this! 🎉

BTW, I vaguely remember to have seen someone mentioned that when both the lowercase and uppercase environment variables exist, like having both npm_config_userconfig and NPM_CONFIG_USERCONFIG set, then one of the two would take precedence due to the way things have been implemented. I might be wrong though, I wonder if there's any plan to clarify that part as well?

Sign up for freeto 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.

[DOCS] npm_config_userconfig vs NPM_CONFIG_USERCONFIG when referred as environment variable

3 participants

@MaxBlack-dev@ryenus@owlstronaut
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

docs: standardize env vars to uppercase convention - #8756

Merged
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case
Nov 25, 2025
Merged

docs: standardize env vars to uppercase convention#8756
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case

Conversation

@MaxBlack-dev

Copy link
Copy Markdown
Contributor

Description

Standardizes all environment variable references in documentation to use uppercase naming convention (NPM_CONFIG_) instead of lowercase (npm_config_).

Changes

  • Changed npm_config_binroot to NPM_CONFIG_BINROOT in scripts.md
  • Changed npm_config_prefix to NPM_CONFIG_PREFIX in npm.md (2 instances)
  • Changed npm_config_ to NPM_CONFIG_ in npm.md
  • Changed npm_config_key to NPM_CONFIG_KEY in npm.md

Fixes

Closes#4891

Rationale

The documentation had inconsistent casing for environment variables - some references used uppercase (NPM_CONFIG_USERCONFIG) while others used lowercase (npm_config_prefix). Following the standard convention, environment variables should always be referenced in uppercase. While npm internally converts both to lowercase, using uppercase in documentation follows the de-facto standard and avoids confusion.

Type of Change

Documentation update

@MaxBlack-dev
MaxBlack-dev requested a review from a team as a code ownerNovember 19, 2025 23:19
@owlstronaut
owlstronaut merged commit 17e154c into npm:latestNov 25, 2025
8 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Nov 25, 2025
@ryenus

ryenus commented Nov 26, 2025

Copy link
Copy Markdown

Wow, finally, thank you @MaxBlack-dev so much for this! 🎉

BTW, I vaguely remember to have seen someone mentioned that when both the lowercase and uppercase environment variables exist, like having both npm_config_userconfig and NPM_CONFIG_USERCONFIG set, then one of the two would take precedence due to the way things have been implemented. I might be wrong though, I wonder if there's any plan to clarify that part as well?

Sign up for freeto 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.

[DOCS] npm_config_userconfig vs NPM_CONFIG_USERCONFIG when referred as environment variable

3 participants

@MaxBlack-dev@ryenus@owlstronaut
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

docs: standardize env vars to uppercase convention - #8756

Merged
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case
Nov 25, 2025
Merged

docs: standardize env vars to uppercase convention#8756
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case

Conversation

@MaxBlack-dev

Copy link
Copy Markdown
Contributor

Description

Standardizes all environment variable references in documentation to use uppercase naming convention (NPM_CONFIG_) instead of lowercase (npm_config_).

Changes

  • Changed npm_config_binroot to NPM_CONFIG_BINROOT in scripts.md
  • Changed npm_config_prefix to NPM_CONFIG_PREFIX in npm.md (2 instances)
  • Changed npm_config_ to NPM_CONFIG_ in npm.md
  • Changed npm_config_key to NPM_CONFIG_KEY in npm.md

Fixes

Closes#4891

Rationale

The documentation had inconsistent casing for environment variables - some references used uppercase (NPM_CONFIG_USERCONFIG) while others used lowercase (npm_config_prefix). Following the standard convention, environment variables should always be referenced in uppercase. While npm internally converts both to lowercase, using uppercase in documentation follows the de-facto standard and avoids confusion.

Type of Change

Documentation update

@MaxBlack-dev
MaxBlack-dev requested a review from a team as a code ownerNovember 19, 2025 23:19
@owlstronaut
owlstronaut merged commit 17e154c into npm:latestNov 25, 2025
8 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Nov 25, 2025
@ryenus

ryenus commented Nov 26, 2025

Copy link
Copy Markdown

Wow, finally, thank you @MaxBlack-dev so much for this! 🎉

BTW, I vaguely remember to have seen someone mentioned that when both the lowercase and uppercase environment variables exist, like having both npm_config_userconfig and NPM_CONFIG_USERCONFIG set, then one of the two would take precedence due to the way things have been implemented. I might be wrong though, I wonder if there's any plan to clarify that part as well?

Sign up for freeto 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.

[DOCS] npm_config_userconfig vs NPM_CONFIG_USERCONFIG when referred as environment variable

3 participants

@MaxBlack-dev@ryenus@owlstronaut
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

docs: standardize env vars to uppercase convention - #8756

Merged
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case
Nov 25, 2025
Merged

docs: standardize env vars to uppercase convention#8756
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case

Conversation

@MaxBlack-dev

Copy link
Copy Markdown
Contributor

Description

Standardizes all environment variable references in documentation to use uppercase naming convention (NPM_CONFIG_) instead of lowercase (npm_config_).

Changes

  • Changed npm_config_binroot to NPM_CONFIG_BINROOT in scripts.md
  • Changed npm_config_prefix to NPM_CONFIG_PREFIX in npm.md (2 instances)
  • Changed npm_config_ to NPM_CONFIG_ in npm.md
  • Changed npm_config_key to NPM_CONFIG_KEY in npm.md

Fixes

Closes#4891

Rationale

The documentation had inconsistent casing for environment variables - some references used uppercase (NPM_CONFIG_USERCONFIG) while others used lowercase (npm_config_prefix). Following the standard convention, environment variables should always be referenced in uppercase. While npm internally converts both to lowercase, using uppercase in documentation follows the de-facto standard and avoids confusion.

Type of Change

Documentation update

@MaxBlack-dev
MaxBlack-dev requested a review from a team as a code ownerNovember 19, 2025 23:19
@owlstronaut
owlstronaut merged commit 17e154c into npm:latestNov 25, 2025
8 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Nov 25, 2025
@ryenus

ryenus commented Nov 26, 2025

Copy link
Copy Markdown

Wow, finally, thank you @MaxBlack-dev so much for this! 🎉

BTW, I vaguely remember to have seen someone mentioned that when both the lowercase and uppercase environment variables exist, like having both npm_config_userconfig and NPM_CONFIG_USERCONFIG set, then one of the two would take precedence due to the way things have been implemented. I might be wrong though, I wonder if there's any plan to clarify that part as well?

Sign up for freeto 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.

[DOCS] npm_config_userconfig vs NPM_CONFIG_USERCONFIG when referred as environment variable

3 participants

@MaxBlack-dev@ryenus@owlstronaut
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

docs: standardize env vars to uppercase convention - #8756

Merged
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case
Nov 25, 2025
Merged

docs: standardize env vars to uppercase convention#8756
owlstronaut merged 1 commit into
npm:latestfrom
MaxBlack-dev:docs/4891-env-var-case

Conversation

@MaxBlack-dev

Copy link
Copy Markdown
Contributor

Description

Standardizes all environment variable references in documentation to use uppercase naming convention (NPM_CONFIG_) instead of lowercase (npm_config_).

Changes

  • Changed npm_config_binroot to NPM_CONFIG_BINROOT in scripts.md
  • Changed npm_config_prefix to NPM_CONFIG_PREFIX in npm.md (2 instances)
  • Changed npm_config_ to NPM_CONFIG_ in npm.md
  • Changed npm_config_key to NPM_CONFIG_KEY in npm.md

Fixes

Closes#4891

Rationale

The documentation had inconsistent casing for environment variables - some references used uppercase (NPM_CONFIG_USERCONFIG) while others used lowercase (npm_config_prefix). Following the standard convention, environment variables should always be referenced in uppercase. While npm internally converts both to lowercase, using uppercase in documentation follows the de-facto standard and avoids confusion.

Type of Change

Documentation update

@MaxBlack-dev
MaxBlack-dev requested a review from a team as a code ownerNovember 19, 2025 23:19
@owlstronaut
owlstronaut merged commit 17e154c into npm:latestNov 25, 2025
8 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Nov 25, 2025
@ryenus

ryenus commented Nov 26, 2025

Copy link
Copy Markdown

Wow, finally, thank you @MaxBlack-dev so much for this! 🎉

BTW, I vaguely remember to have seen someone mentioned that when both the lowercase and uppercase environment variables exist, like having both npm_config_userconfig and NPM_CONFIG_USERCONFIG set, then one of the two would take precedence due to the way things have been implemented. I might be wrong though, I wonder if there's any plan to clarify that part as well?

Sign up for freeto 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.

[DOCS] npm_config_userconfig vs NPM_CONFIG_USERCONFIG when referred as environment variable

3 participants

@MaxBlack-dev@ryenus@owlstronaut