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

Meta: use new WHATWG boilerplate and license #118

Merged
merged 1 commit into from
Jan 5, 2018
Merged

Conversation

domenic
Copy link
Member

@domenic domenic commented Jan 4, 2018

Do not merge until speced/bikeshed#1155 is merged.


Preview | Diff

!Commits: <a href="https://twitter.com/notifyapi">@notifyapi</a>
!Tests: <a href=https://github.com/w3c/web-platform-tests/tree/master/notifications>web-platform-tests notifications/</a> (<a href=https://github.com/w3c/web-platform-tests/labels/notifications>ongoing work</a>)
!Translation (non-normative): <span title="Simplified Chinese"><a href=https://w3c-html-ig-zh.github.io/notifications/whatwg/ lang=zh hreflang=zh rel=alternate>简体中文</a></span>
Translation: zh-cn https://w3c-html-ig-zh.github.io/notifications/whatwg/
Copy link
Member

Choose a reason for hiding this comment

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

Is a more specific code better? (The resource itself seems to declare "en"...)

Copy link
Member Author

@domenic domenic Jan 4, 2018

Choose a reason for hiding this comment

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

In this case yes, as zh-cn is specifically Simplified Chinese (which the resource is). It might be good to file an issue on the resource to update its language code, indeed.

Copy link
Member

Choose a reason for hiding this comment

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

https://github.com/w3c-html-ig-zh/notifications has no way to file issues (and also seems like it's not actively maintained).

Copy link
Member Author

Choose a reason for hiding this comment

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

Should we remove it?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll leave it for now, on the logic that separate translation changes should stay separate.

@annevk annevk mentioned this pull request Jan 5, 2018
@domenic domenic merged commit 2dacbc8 into master Jan 5, 2018
@domenic domenic deleted the new-boilerplate branch January 5, 2018 15:31
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