-
Notifications
You must be signed in to change notification settings - Fork 20
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
Update to latest X509 and DNS APIs. #240
Conversation
edf8a3e
to
71e92c8
Compare
Could you rebase this on top of master? Thanks! |
CHANGES.md
Outdated
@@ -32,41 +37,41 @@ | |||
|
|||
### v0.9.0 | |||
|
|||
- Add connection progress indicator (@talex5 #220). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these changes intentional? Trailing space is significant in markdown (forces a linebreak).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@talex5 woops sorry I haven't realised these spaces have semantics. I guess @tmcgilchrist has the same editor configuration as mine which removes trailing spaces automatically.
Do you want me to revert #239 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll re-instate the whitespaces, Emacs tidies up extra whitespace automatically when sometimes it shouldn't.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any chance we could have a release with this change? |
…age and capnp-rpc-lwt (1.2.1) CHANGES: - Fix Fmt deprecation warnings (@tmcgilchrist mirage/capnp-rpc#239). - Update to latest X509 and DNS APIs (@tmcgilchrist mirage/capnp-rpc#240).
…age and capnp-rpc-lwt (1.2.1) CHANGES: - Fix Fmt deprecation warnings (@tmcgilchrist mirage/capnp-rpc#239). - Update to latest X509 and DNS APIs (@tmcgilchrist mirage/capnp-rpc#240).
On top of #239