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

Don't export character as an alias of code point #450

Closed
wants to merge 1 commit into from

Conversation

fantasai
Copy link

@fantasai fantasai commented May 5, 2022

“Character” is an ambiguous term and is used in multiple senses in the Web platform. Don't alias it to only one of them. #449


Preview | Diff

“Character” is an ambiguous term and is used in multiple senses in the Web platform. Don't alias it to only one of them. whatwg#449
@fantasai
Copy link
Author

fantasai commented May 5, 2022

OK, so:

  • The build claims to be failing because “character” can be referencing that term's definition in two different specs, css-text or infra, but the pull request is in fact removing the second definition (afaict; that is the point of this PR).
  • I don't know in what capacity I'm supposed to sign the participation agreement, it's not clear that any of the categories applies to me.

Feel free to close the PR and make the edits on your own, I don't know how to untangle any of this.

@domenic
Copy link
Member

domenic commented May 5, 2022

  • I don't know in what capacity I'm supposed to sign the participation agreement, it's not clear that any of the categories applies to me.

From what I understand you work as a contractor in the field of web technologies so should be covered under one or more of the entities you're contracting for.

Feel free to close the PR and make the edits on your own, I don't know how to untangle any of this.

Given that the discussion in #449 is still ongoing, I'll close for now.

@domenic domenic closed this May 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants