-
Notifications
You must be signed in to change notification settings - Fork 64
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
[wg/did] change suggestion by @rxgrant #468
Conversation
9fd1267
to
b855c3d
Compare
in response to @peacekeeper #448 (comment)
b855c3d
to
dead3ea
Compare
I would change the "should not"s to "MUST NOT"s. |
d250bd3
to
aba1747
Compare
Co-authored-by: Ivan Herman <[email protected]>
STRAWPOLL : please react to this post ("this PR" refers to changes up to e4ed7af)
|
I don't understand the goal of obviating this. |
@cwilso it was explained in the thread that this came out of. |
I continue to maintain that this is an unnecessary change. |
@rxgrant that's... not a response? Where was that discussion? |
@brentzundel see #467 (comment)
That being said, as you +1'ed on Joe's proposal above, I'm accepting it. |
Co-authored-by: Joe Andrieu <[email protected]>
I believe that your latest proposal ("beyond those explicitly listed") is enough to "protect" the registry. |
This PR reflects a change orinally proposed by @rxgrant at #448 (comment) .
It was discussed with some AC reviewers during today's meeting.