Skip to content
This repository has been archived by the owner on Jun 13, 2024. It is now read-only.

fix: start proxy in cjs #22

Merged
merged 1 commit into from
Apr 29, 2023
Merged

fix: start proxy in cjs #22

merged 1 commit into from
Apr 29, 2023

Conversation

fubhy
Copy link
Collaborator

@fubhy fubhy commented Apr 29, 2023

PR-Codex overview

This PR updates the http-proxy import in createProxy.ts file to be compatible with CommonJS (cjs) environments. It also adds new tests to check the functionality of createPool() and startProxy() functions.

Detailed summary

  • Fixed import from http-proxy package in cjs environments.
  • Added tests to check the functionality of createPool() and startProxy() functions.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@changeset-bot
Copy link

changeset-bot bot commented Apr 29, 2023

🦋 Changeset detected

Latest commit: d142ad1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@viem/anvil Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@commit-lint
Copy link

commit-lint bot commented Apr 29, 2023

Bug Fixes

Contributors

fubhy

Commit-Lint commands

You can trigger Commit-Lint actions by commenting on this PR:

  • @Commit-Lint merge patch will merge dependabot PR on "patch" versions (X.X.Y - Y change)
  • @Commit-Lint merge minor will merge dependabot PR on "minor" versions (X.Y.Y - Y change)
  • @Commit-Lint merge major will merge dependabot PR on "major" versions (Y.Y.Y - Y change)
  • @Commit-Lint merge disable will desactivate merge dependabot PR
  • @Commit-Lint review will approve dependabot PR
  • @Commit-Lint stop review will stop approve dependabot PR

@fubhy fubhy merged commit f03b3ec into main Apr 29, 2023
@fubhy fubhy deleted the fix/cjs-start-proxy branch April 29, 2023 10:35
@github-actions
Copy link
Contributor

Size Change: -9 B (0%)

Total Size: 13.6 kB

Filename Size Change
packages/anvil.js/dist/cjs/proxy/createProxy.js 1.31 kB -3 B (0%)
packages/anvil.js/dist/esm/proxy/createProxy.js 1.25 kB -6 B (0%)
ℹ️ View Unchanged
Filename Size
packages/anvil.js/dist/cjs/anvil/createAnvil.js 1.61 kB
packages/anvil.js/dist/cjs/anvil/getVersion.js 324 B
packages/anvil.js/dist/cjs/anvil/stripColors.js 327 B
packages/anvil.js/dist/cjs/anvil/toArgs.js 405 B
packages/anvil.js/dist/cjs/anvil/toFlagCase.js 282 B
packages/anvil.js/dist/cjs/index.js 355 B
packages/anvil.js/dist/cjs/pool/createPool.js 906 B
packages/anvil.js/dist/cjs/proxy/fetchLogs.js 408 B
packages/anvil.js/dist/cjs/proxy/parseRequest.js 495 B
packages/anvil.js/dist/cjs/proxy/startProxy.js 717 B
packages/anvil.js/dist/cjs/types.js 124 B
packages/anvil.js/dist/esm/anvil/createAnvil.js 1.52 kB
packages/anvil.js/dist/esm/anvil/getVersion.js 257 B
packages/anvil.js/dist/esm/anvil/stripColors.js 260 B
packages/anvil.js/dist/esm/anvil/toArgs.js 334 B
packages/anvil.js/dist/esm/anvil/toFlagCase.js 216 B
packages/anvil.js/dist/esm/index.js 165 B
packages/anvil.js/dist/esm/pool/createPool.js 815 B
packages/anvil.js/dist/esm/proxy/fetchLogs.js 338 B
packages/anvil.js/dist/esm/proxy/parseRequest.js 436 B
packages/anvil.js/dist/esm/proxy/startProxy.js 638 B
packages/anvil.js/dist/esm/types.js 64 B

compressed-size-action

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.

1 participant