-
Notifications
You must be signed in to change notification settings - Fork 22.6k
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
Rationalize top-level Web API pages #1579
Comments
…les (mdn#1579) * configure eslint * fix eslint errors * rename class * add missing curly braces * add missing empty line * allow one line declaration in specific file * allow missing block after for * enforce eol as last character * enforce template literals * enforce no-var eslint rule * enforce prefer-arrow-callback eslint rule * enforce comma-spacing eslint rule * enforce comma-style eslint rule * enforce func-call-spacing eslint rule * enforce key-spacing eslint rule * enforce keyword-spacing eslint rule * enforce no-multiple-empty-lines eslint rule * enforce no-trailing-spaces eslint rule * enforce space-before-function-paren eslint rule * enforce indentation within switch statements * enforce 1tbs brace-style with allowSingleLine option * enforce no-array-constructor eslint rule * enforce no-new-object eslint rule * enforce space-infix-ops eslint rule * enforce no-else-return eslint rule * add styleguide check to travis * add syntax highlight to the contributing md * adjust styleguide in the contributing md * remove osolated examples from the contribution guidelines * remove unnecessary empty lines from contribution guidelines * add override example to the contribution guidelines * remove outdated styleguide hints
As this one is nearing 2 years without any updated, I suggest we consider closing it and maybe instead perhaps adding to the OWD work list (if it’s not already included there) — following the general principle that it’s good to not keep issues open indefinitely without any associated updates to the content. |
Fair enough. I still think this is worth doing but it's obsolete at least in that it talks about using tags rather than page types, and it might (I hope) be obsoleted by openwebdocs/project#76, so I'm less invested in getting it done. |
According to mdn/sprints#3327 we have ~1100 pages directly under Web/API.
I believe that every page directly under Web/API must be one of:
While it's hard to tell which pages are in group (1), a reasonable heuristic seems to be: if the title contains a space, the page is an API overview page. But that heuristic we have about 100 API overview pages, leaving about 1000 others.
As far as I know, all the others ought to be either interface pages or dictionary pages, so ought to be listed in GroupData.json. In fact, ~250 of these pages are not listed in GroupData.
Additionally, some interfaces/dictionaries appear under more than one group in GroupData. I believe that this should not be allowed.
So we should:
The text was updated successfully, but these errors were encountered: