Skip to content
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

Remove normative text in informative section. #84

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

msporny
Copy link
Member

@msporny msporny commented Aug 31, 2024

This PR is an attempt to address issue #46 by removing normative language from an informative section.

This PR specifically does not make the Terminology section normative since the terminology section does not contain any normative language.


Preview | Diff

@selfissued
Copy link
Collaborator

I'm fine with this change, but it does not address #46. See @jyasskin's comment at #46 (comment).

@msporny msporny added the normative This item is a normative change. label Sep 2, 2024
@iherman
Copy link
Member

iherman commented Sep 4, 2024

The issue was discussed in a meeting on 2024-09-04

  • no resolutions were taken
View the transcript

2.3. Remove normative text in informative section. (pr controller-document#84)

See github pull request controller-document#84.

Brent Zundel: changes on 84 are straightforward, so this can be brief.

Manu Sporny: agreed.

@msporny
Copy link
Member Author

msporny commented Sep 6, 2024

Normative, multiple reviews, no changes requested, no objections, merging.

@msporny msporny merged commit 2b1587c into main Sep 6, 2024
1 check passed
@msporny msporny deleted the msporny-nonnorm-should branch September 6, 2024 21:11
@TallTed
Copy link
Member

TallTed commented Sep 9, 2024

I suggest s/in informative/from informative/ in title.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
normative This item is a normative change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants