Skip to content

Conversation

mturkia
Copy link
Contributor

@mturkia mturkia commented Oct 22, 2021

Description

Additional documentation. These instructions lower the disk usage of sentries to 20% of the
default configuration. Some of these configurations can safely be used also on a validator.


For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

These instructions lower the disk usage of sentries to 20% of the
default configuration. Some of these configurations can safely be used
also on a validator.
@kinrokinro
Copy link
Contributor

Great, good job!

Copy link
Collaborator

@fedekunze fedekunze left a comment

Choose a reason for hiding this comment

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

Thanks @mturkia! Can you drop your evmos address on the comments?

@fedekunze fedekunze merged commit a601428 into evmos:main Oct 22, 2021
@mturkia
Copy link
Contributor Author

mturkia commented Oct 22, 2021

Thanks @mturkia! Can you drop your evmos address on the comments?

evmos1zk8ef40wrhm8z9sqcmfa0zhz8euyyra8lmuesf

faddat referenced this pull request in notional-labs/evmos Dec 9, 2021
* evm: fix sig verification

* fixes

* test fixes
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.

3 participants