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

Contributing #469

Closed
wants to merge 61 commits into from
Closed

Contributing #469

wants to merge 61 commits into from

Conversation

brapse
Copy link
Contributor

@brapse brapse commented Jul 22, 2020

We need a minimal CONTRIBUTING file for #323.
This ports over most of the one from tendermint and is probably missing a bunch of stuff but is hopefully a start.

  • Referenced an issue explaining the need for the change
  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGES.md

* refactor produce_validator to allow calling it from within Rust code,
to easily produce Info struct
* add --ignore-stdin option to allow skipping parsing from STDIN
@brapse
Copy link
Contributor Author

brapse commented Jul 22, 2020

This was a mistake, wrong base branch. 🤦

@brapse brapse closed this Jul 22, 2020
@brapse brapse deleted the brapse/initial-contributing branch July 22, 2020 10:06
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