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

[wording] no restrictions from supported concept definition #518

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

dginev
Copy link
Contributor

@dginev dginev commented Nov 12, 2024

Following #515 and the WG meeting on 11/07/2024, we had an offline discussion with @polx on some minimal text that avoids a strict reading of the "supported concept" definition.

This PR contains our proposal.

Copy link
Collaborator

@davidcarlisle davidcarlisle left a comment

Choose a reason for hiding this comment

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

Personally I don't think this makes things that much clearer, but more than one person thought the existing text wasn't that clear and this addition seems fine to me so happy to add this. I would have just merged as editorial but since it has been the subject of some WG debate I'll just approve and leave time for Neil to review.

Copy link
Contributor

@NSoiffer NSoiffer left a comment

Choose a reason for hiding this comment

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

I agree with David that this adds nothing new. Nonetheless, since two people expressed confusion and David doesn't object, I'll approve this in the hopes that this makes the rec clearer to some and do the merge.

@NSoiffer NSoiffer merged commit 972e052 into w3c:main Nov 14, 2024
2 of 3 checks passed
@dginev dginev deleted the does-not-exclude-implementations branch November 14, 2024 02:12
github-actions bot added a commit that referenced this pull request Nov 14, 2024
SHA: 972e052
Reason: push, by NSoiffer

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants