Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Npm #39

Merged
merged 98 commits into from
Apr 26, 2024
Merged

Npm #39

merged 98 commits into from
Apr 26, 2024

Conversation

makoto
Copy link
Member

@makoto makoto commented Apr 18, 2024

This PR makes the following changes

  • Move crosschain-resolver and crosschain-reverse-resolver to https://github.com/ensdomains/ens-evmgateway
  • Publish evm-gateway, evm-verifier, l1-gateway, l1-verifier as npm modules.
  • Add logger (worker.ts script currently has some redundant code which @mdtanrikulu will handle with different PR)
  • Add Sepolia deployment detail of OP, Base, and Arb

@makoto makoto mentioned this pull request Apr 18, 2024
Closed
op-gateway/src/worker.ts Outdated Show resolved Hide resolved
arb-verifier/test/testArbVerifier.ts Outdated Show resolved Hide resolved
arb-gateway/src/worker.ts Outdated Show resolved Hide resolved
op-verifier/deploy_l1/00_op_verifier.ts Outdated Show resolved Hide resolved
op-verifier/deploy_l1/00_op_verifier.ts Outdated Show resolved Hide resolved
Copy link
Member

@Arachnid Arachnid left a comment

Choose a reason for hiding this comment

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

I see some redundant logging boilerplate being added to each gateway. Can that be abstracted into common code?

@mdtanrikulu
Copy link
Member

mdtanrikulu commented Apr 19, 2024

I see some redundant logging boilerplate being added to each gateway. Can that be abstracted into common code?

I work on this in another branch. moved the logResult into server-analytics library.

makoto and others added 6 commits April 19, 2024 12:57
* remove log result util

* bump server-analytics version, update types and functions

* bypass frozen-lockfile flag for deps update

* bypass frozen-lockfile flag for deps update

* lint

* revert frozen-lockfile flag

* keep  rozen-lockfile disabled

* lint
op-gateway/README.md Outdated Show resolved Hide resolved
op-gateway/src/worker.ts Outdated Show resolved Hide resolved
makoto and others added 2 commits April 26, 2024 10:50
Co-authored-by: Nick Johnson <arachnid@notdot.net>
* move propsDecoder

* Add Request as type

* Update workers-types

* Fix lint error
@makoto makoto merged commit 588d6b5 into main Apr 26, 2024
5 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.

5 participants