-
Notifications
You must be signed in to change notification settings - Fork 0
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
Repository Initialization #1
Conversation
62e22d0
to
e62ea1c
Compare
e62ea1c
to
93ea82e
Compare
Basic Github Action to enforce style and test on pull requests
e99388c
to
79b6c40
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't gone into deep but the overall structure looks great! 🚀
84c8352
to
45a509f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just left some questions
@@ -0,0 +1,61 @@ | |||
{ | |||
"name": "excubiae", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there's no need for a version on the root package.json
since is a monorepo.
@@ -0,0 +1,10 @@ | |||
# core | |||
DEFAULT_NETWORK=hardhat |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
where's this used? iirc in hardhat the hardhat network is the default already
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this can be helpful for custom default networks (running nodes different from current hardhat environment / network).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh right thought it'd be more convenient to pass --network $NETWORK to the command so you don't have to switch env var but this works too
@@ -0,0 +1,30 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why two package.json in this package?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this makes easy to release just the contracts and avoid carrying on a huge bundle of configs & other dev env packages. This is the default project structure when using hardhat
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's go!!! 🚀
The idea here is to get the Excubiae project up and running with a basic boilerplate to test the configs. As a general rule, we look to the best-maintained and best-configured TS projects in PSE as guidance. Some examples include semaphore and zk-kit.
The
contracts
package supports bothhardhat
andfoundry
. Therefore, thecontracts
package needs to be renamed for the necessaryremappings
, and so some configs have been moved around to minimise duplication.