## Description
This enables the auto-formatting to automatically format and wrap
comments so they do not exceed the length of the code.
This is still an experimental feature it seems, but we've had good
results with the import ordering so this might work out as well.
## Breaking Changes
none
## Notes & open questions
This is quite the bloodbath, probably should have been
expected. Partially I wish we could enable this only for new code but
hey.
I've fixed up the worst things, let me know if there's still anything
else that needs fixing up.
I'd secretly like to set this to 92 rather than 100. But I guess that
would be pushing my personal preferences too far.
It this works we should probably also do this on iroh.
I love the irony that the line in the Makefile.toml is now well over
100 characters long.
I'd prefer to only merge this by consensus, vetos are also accepted.
## Change checklist
- [x] Self-review.
- [x] Documentation updates following the [style
guide](https://rust-lang.github.io/rfcs/1574-more-api-documentation-conventions.html#appendix-a-full-conventions-text),
if relevant.
## Description
This updates semver checks with the following behaviour:
- The baseline for the check is now the latest published release on
crates.io
- If you make a semver-breaking change you must also bump the package
version number so that the semver check will pass. Because we like
to bump the versions of all crates at the same time you can not use
pre-releases since cargo does not allow mixing those and the iroh
patchbay tests has 2nd-level dependencies on noq-udp (via netwatch).
- Adding deprecated items is allowed in minor version bumps, matching
https://semver.org/#how-should-i-handle-deprecating-functionality
- Bump noq-proto version because we previously added deprecations.
- Now the baseline is somewhat stable, enable caching.
## Breaking Changes
none
## Notes & open questions
I tested this first by not bumping the noq-proto version number and it
fails in that case. Which is due to #725.
Once this is merged I will make the semver check required in noq.
The version bumping required to make semver-checks pass could be a bit
annoying. It will also make patching iroh for noq a little bit harder.
As shown by what the patchbay check has to do now. I'm tempted to think
for now that this is worth it, but happy to think about how to tweak
this as we gain experience.
OTOH having to bump to the right version means that come to a release we
do know what the next version should be. Which is probably good.
## Change checklist
- [x] Self-review.
`cargo release` command successfully published the crates, but the
commit created by `cargo release` to update the version numbers was
rejected, because I don't have permissions to push to the `main` branch
directly.
These are the appropriate changes to ensure the version numbers in our
code align with the released crates.
Floris Bruynooghe