-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
ko: Convert noteblocks to GFM Alerts (part 1) #22758
Conversation
This PR converts the noteblocks for the Korean locale to GFM Alerts syntax, using a [conversion script](https://github.com/queengooborg/mdn-toolkit/blob/main/upgrade-noteblock.js). This is part 1. Note: manual adjustments have also been made to correct some issues, including capitalization, syntax, duplicated keywords and more.
Preview URLs (100 pages)
Flaws (7127)Note! 16 documents with no flaws that don't need to be listed. 🎉 URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
No first query argument or 'browser-compat' or 'spec-urls' front-matter value passed`
No first query argument or 'browser-compat' front-matter value passed`
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
External URLs (69)URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
(comment last updated: 2024-08-08 08:19:52) |
@queengooborg Hello. Contributors are slow to respond to existing PRs(and there is too much PR. We want to merge as much as possible.), so we plan to merge them first and then merge the current noteblocks-related PRs after requesting a batch resolution on Wednesday. I kindly ask for your understanding. |
I can totally understand wanting to work through the existing backlog first before performing this conversion! There's no rush to merge this right away -- it isn't a blocker on any Yari updates or other things, and is mainly for convenience purposes. I'll keep these PRs open for when you/the Korean team is ready to review and merge them, and naturally, will fix any merge conflicts along the way! |
This PR converts the noteblocks for the Korean locale to GFM Alerts syntax, using a conversion script. This is part 1. Note: manual adjustments have also been made to correct some issues, including capitalization, syntax, duplicated keywords and more.