-
Notifications
You must be signed in to change notification settings - Fork 22
feat: ADR-004 (Nakamoto Bonus) #153
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
Conversation
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 personally would prefer to have two separate PRs
- add fork of distribution module
- add code for nakamoto bonus
As it is right now, it's very hard to review: it's unclear what is old and what is new
ifneq ($(GO_SYSTEM_VERSION), $(GO_REQUIRED_VERSION)) | ||
@echo 'ERROR: Go version $(GO_REQUIRED_VERSION) is required for building AtomOne' | ||
ifeq ($(shell echo "$(GO_SYSTEM_VERSION) $(GO_REQUIRED_VERSION)" | awk '{if ($$1 >= $$2) print "1"}'),) | ||
@echo 'ERROR: Go version $(GO_REQUIRED_VERSION) or greater is required for building AtomOne (curent: $(GO_SYSTEM_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.
Please revert this. We use exact versions to achieve reproducible builds.
See https://github.com/atomone-hub/atomone?tab=readme-ov-file#reproducible-builds
closing to address my comment. PR for distribution module is #181, while PR for the Nakamoto bonus will come later. |
implementation of https://github.com/atomone-hub/atomone/blob/main/docs/architecture/adr-004-nakamoto-bonus.md
needs merge first: #141