-
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
chore(ia): Tidy MDN meta dir #37784
chore(ia): Tidy MDN meta dir #37784
Conversation
- [rari](https://github.com/mdn/rari) | ||
- : This is the backend part of the MDN Web Docs platform, where you'll go if you want to make changes to page structures and rendering machinery. | ||
- [yari](https://github.com/mdn/yari) | ||
- : This is the frontend part of the MDN Web Docs platform, where you'll find styles, design, and layout functionality. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO these are infrastructure, even if they are main repositories, they shoud be there because most consumers of this document need to know very little about them.
|
||
- : These guides help you get started with contributing to MDN Web Docs. They also cover topics such as how you can help with tasks and issues, open discussions, and suggest content. If you need help or want to contact us, you'll find the information here. | ||
**MDN Web Docs** is a free-to-use resource documenting open web technologies. | ||
Our mission is to provide developers with the information they need to easily build projects on the web platform. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is "the web platform"? We use the term a lot, and I think we kind of have a somewhat clear definition of that that means. But it feels like a term that should be linked. Anyway, just rambling now, this is a huge PR to review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, we should explain that.
this is a huge PR to review.
Sorry, and thank you! :)
files/en-us/mdn/writing_guidelines/page_structures/page_types/index.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bsmth I've finished looking. Can't guarantee the review was thorough, but I did look at all the pages. I always worry that as reviewer I'll be the last person that reads this kind of docs, but I guess we have to have it :-0
Anyway, these changes look sane, in particular the deletion of some of the cruft and the naming consistency work.
Co-authored-by: Hamish Willee <[email protected]>
Thank you very much 🙏🏻 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Thanks :) Just a tiny nitpick.
files/en-us/mdn/writing_guidelines/howto/document_an_http_header/index.md
Outdated
Show resolved
Hide resolved
…er/index.md Co-authored-by: Vadim Makeev <[email protected]>
Thanks, all! Merging now! |
Description
Notable: pages worth checking prose:
files/en-us/mdn/index.md
files/en-us/mdn/community/index.md
files/en-us/mdn/community/open_source_etiquette/index.md
files/en-us/mdn/community/our_repositories/index.md
Changes:
rari content fmt-sidebars
files/en-us/mdn/
changes update cross-references to moved filesMDN/Changelog
->MDN/Writing_guidelines/Changelog
Removals:
Motivation
Docs IA & maintenance