fix: noproxy (#3259)

close #3258

.changeset/ten-lions-complain.md

@@ -0,0 +1,5 @@
+---
+"@pnpm/config": patch
+---
+
+The noproxy setting should work.

packages/config/src/index.ts

@@ -390,7 +390,7 @@ export default async (
     pnpmConfig.httpProxy = pnpmConfig.httpsProxy ?? getProcessEnv('http_proxy') ?? getProcessEnv('proxy')
   }
   if (!pnpmConfig.noProxy) {
-    pnpmConfig.noProxy = getProcessEnv('no_proxy')
+    pnpmConfig.noProxy = pnpmConfig['noproxy'] ?? getProcessEnv('no_proxy')
   }
   pnpmConfig.enablePnp = pnpmConfig['nodeLinker'] === 'pnp'
 

packages/config/test/index.ts

@@ -656,3 +656,16 @@ test('warn user unknown settings in npmrc', async () => {
 
   expect(noWarnings).toStrictEqual([])
 })
+
+test('getConfig() converts noproxy to noProxy', async () => {
+  const { config } = await getConfig({
+    cliOptions: {
+      noproxy: 'www.foo.com',
+    },
+    packageManager: {
+      name: 'pnpm',
+      version: '1.0.0',
+    },
+  })
+  expect(config.noProxy).toBe('www.foo.com')
+})