Release 0.21.0

[apoelstra]

.

[sanket1729]

This breaks a lot of things downstream. I think it might a good idea to highlight a few of those in release notes.
Renamed:

1. Moved schnorrsig module to schnorr
2. Moved schnorrsig::PublicKey to XOnlyPublicKey
3. Moved recovery to ecdsa::recovery.

Should I add a PR making these non-breaking changes?

[apoelstra]

@sanket1729 you mean, PR to modify the changelog? I can do it.

[sanket1729]

@apoelstra , see #348 . Would be great to have that in because downstream libs rust-bitcoin raise many compile errors.

[apoelstra]

Great idea @sanket1729. Merged.

[sanket1729]

utACK 8cfd26a

[elichai]

ACK 8cfd26a Checked the changelog against the git log

[apoelstra]

I'd like to get #344 in first actually, it's an API-breaking change but a very minor one.

[apoelstra]

Rebased and added note to changelog.

[sanket1729]

utACK c3df9b8. Excited for the new release

[apoelstra]

@elichai can I get a quick re-ACK and then I'll do it?

[elichai]

re-ACK c3df9b8

[apoelstra]

Merged. Published.

Need to push new point release of secp256k1-sys because of #345 ... and actually I think #345 is missing a pub qualifier :)

[sanket1729]

Might also need a point release for rust-secp. See #360

[sanket1729]

utACK c3df9b8. Excited for the new release

Should have tested with rust-bitcoin before ACKing

[apoelstra]

Lol oops. I had also wanted to get #353 in (though less critically). We'll do a point release.