Rework README to point users to rustls-platform-verifier#141
Conversation
db4802e to
65c39f4
Compare
cpu
left a comment
There was a problem hiding this comment.
I like the fancy markdown rendering of the important note 👍
|
I'm fine with this as an alternative to my suggested changes! I was hesitant to do it myself since I thought we were not quite ready to make the recommendation cutover but I have no issues with the idea. |
Any particular reason? IMO this note is unlikely to affect existing users much and we already have some adoption (for example in Quinn, and mainline rustup). |
|
I had this comment in mind, specifically this part:
The new wording here is basically a soft-deprecation of the crate. |
|
Right. I think the deprecation should be more of a slightly lagging indicator whereas advice from this README seeks to be leading. We've also made some progress since then with the additivity issue. |
|
That sounds fine to me 👍 . |
|
I've fixed the wording as pointed out by @ctz and split this in more commits to make it clearer why things changed. Please have another look! |
Alternative to #140, which IMO is too cautious.
cc @complexspaces
(The screenshot is mostly for showing the way it renders -- review the diff for current text.)