-
Notifications
You must be signed in to change notification settings - Fork 998
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
Move deposit contract back to this repo #980
Comments
Initially (~month ago) I was looking for solutions to have the generators/tests live elsewhere. They are just too dependent/close with the spec itself, so it made sense to go for a more monorepo-ish approach. We could take it further, and just call this the "eth2" monorepo, with release & unified testing. I'm ok with more things being merged into this repo, but we should also look into alternatives, if there are any. And the net benefits of course. So, to iterate, current options (with their individual pros/cons) are: deposit contract in external repoPros:
Cons:
merge into this repoPros:
Cons:
Git-submodule one in the otherNot tried before, not sure about net-benefits, not likely to work well. alternatives?Suggestions welcome. |
As a spec consumer, my preference would be for a mono-repo as it would make it easier to locate things and easier to create issues. Additionally, when adding new components its much easier to notice a new dir in an existing repo than a whole new repo. In my experience, when you're dealing with a big, inter-connected project addressing "issues/PRs clutter" by splitting into separate repos means you have just as many issues/PRs but they're now harder to locate ("which repo was that in again?") or in the wrong place ("this issue should really be on repo X"). Monorepo preference is obviously subjective, just adding my thoughts. |
I'm pro bringing it back |
Deposit contract was moved to another repo because we wanted to make eth2.0-specs pure with just documents. But after some recent updates (thanks to @protolambda and @djrtwo), now
eth2.0-specs
repo is the CI/test-gen center. It seems moving deposit contract back can make the spec-syncing and release easier.The text was updated successfully, but these errors were encountered: