consensus::Params should be non_exhaustive

[hrouis]

fixes: #1213

[apoelstra]

ACK 445037f

[tcharding]

Thanks for the contribution @hrouis, we appreciate your efforts. Can I offer a couple of high level comments

• If you put fixes: #1213 in the PR description github will autoclose the issue for us.

There are a bunch of other keywords one can use instead of 'fixes', here is a list https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests

• There is a nice blogpost on writing good clean commit messages, I like to share it with folks. PR/commit messages help reviewers review your work more easily and as you do more involved patches will help your work get merged more easily.

https://cbea.ms/git-commit/

Thanks again man!

[tcharding]

ACK 445037f

[hrouis]

Thank you for the pointers and links, will definitely apply them for the next PRs

[Kixunil]

ACK 445037f

I don't mind the commit message here, this is almost as trivial as fixing a typo. :)

[tcharding]

Thank you for the pointers and links, will definitely apply them for the next PRs

Thanks man, hope I didn't come across too nit-picky :)