Skip to content

remove normative language from Security Considerations #231

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

Merged
merged 2 commits into from
Aug 14, 2025
Merged

Conversation

pchampin
Copy link
Contributor

@pchampin pchampin commented Aug 8, 2025

addresses #183

I changed all capitalized BCP14 keywords, as well as some lowercase verbs that could also be mistaken for normative language.

I kept some (lowercase) "must"s and "may"s, when I considered that they were more idiomatic than the alternatives, and that the context made its clear that it was not normative text (e.g. "Care must be taken to...", "avoid IRIs that may look similar").


Preview | Diff

@pchampin pchampin requested review from hartig, gkellogg and afs August 8, 2025 13:16
@pchampin pchampin added the ms:CR Milestone: Candidate Recommendation label Aug 8, 2025
@pchampin
Copy link
Contributor Author

pchampin commented Aug 8, 2025

in response to action w3c/rdf-star-wg#175

Characters in different scripts may look similar (for instance,
a Cyrillic "о" may appear similar to a Latin "o").
A character followed by combining characters may have the same visual representation
<p>Multiple <a>IRIs</a> can have the same appearance.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These last four changes are not needed and, in my opnion, are less readable than the unchanged text.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(this part of the PR isn't in the "preview" or "diff")

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@afs — I see these four lines (2035-2038) in both Preview and Diff. Perhaps try searching for Characters in different? These lines are less obvious in the rendered pages than in the HTML source, because the HTML source has new lines that get folded into spaces in the rendered page.

@pfps — It's easier to know what "last four changes" you're talking about if you drag-select the plus-sign from the first line of concern down to the last line of concern. This also allows for easy suggestion of changes to the source, where I would currently have to start a new thread to insert such suggestion about all 4 of these lines.

Comment on lines +2035 to +2038
<p>Multiple <a>IRIs</a> can have the same appearance.
Characters in different scripts can look similar (for instance,
a Cyrillic &quot;&#1086;&quot; can appear similar to a Latin &quot;o&quot;).
A character followed by combining characters can have the same visual representation
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I support the changes from may to can in these four lines (2035–2038).

Copy link
Member

@gkellogg gkellogg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With @TallTed's suggestions.

Co-authored-by: Ted Thibodeau Jr <[email protected]>
@pchampin pchampin merged commit e9e4799 into main Aug 14, 2025
2 checks passed
@w3cbot
Copy link

w3cbot commented Aug 14, 2025

This was discussed during the #rdf-star meeting on 14 August 2025.

View the transcript

w3c/rdf-concepts#231

<gb> Pull Request 231 remove normative language from Security Considerations (by pchampin) [ms:CR]

pchampin: PR 231 is ready. Related to actions we talked about earlier.
… been approved by editors. I think it's ready to ship.

ora: happy to have you merge it.

<TallTed> I've got ~39 rdf-shapes notifications, plus all the sub-repos, needing review

TallTed: need to ask for more time.

ora: just ping pchampin when you're ready.

TallTed: will put approve on whatever I've gone over.

pchampin: I'd ask editors to merge this after TallTed's approvel as I'll be away until next Friday.

gkellogg: I see TallTed's approval in the review.

TallTed: I don't know which PRs are being discussed. If I approved, then merge it. If things have changed since the approval, please don't.

ora: subtopic: w3c/rdf-n-quads#80

gkellogg: document we're focused on is n-triples, but echos through all turtle-like specs. I've been applying changes.
… will also need similar changes to rdf-tests


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ms:CR Milestone: Candidate Recommendation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants