Skip to content

fix: accept request --no-proxy flag#28

Merged
alexrisch merged 1 commit into
mainfrom
fix/request-no-proxy
Jun 24, 2026
Merged

fix: accept request --no-proxy flag#28
alexrisch merged 1 commit into
mainfrom
fix/request-no-proxy

Conversation

@alexrisch

Copy link
Copy Markdown
Collaborator

Changes

  • Route tempo request --no-proxy around Incur's --no-foo negation parser so it is not treated as proxy=false.
  • Keep the public help/describe spelling as --no-proxy.
  • Add request parser coverage and a patch changelog entry.

Testing

  • pnpm check:format
  • pnpm typecheck
  • pnpm exec vitest run test/request.test.ts
  • pnpm changelog:validate
  • Manual: pnpm exec tsx src/request-cli.ts --no-proxy --dry-run http://example.com

Note: origin/main currently has an unrelated test/cli-describe.test.ts failure because package.json is 0.6.2 while src/shared/constants.ts defaults to 0.6.1.

@alexrisch alexrisch force-pushed the fix/request-no-proxy branch from 2a49860 to ff7def4 Compare June 24, 2026 00:48
@alexrisch alexrisch merged commit 9cc1216 into main Jun 24, 2026
4 checks passed
Sign up for free to 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.

1 participant