This document describes the review process for content changes on MDN Web Docs, and is for use by those who have been tasked with reviewing MDN content PRs.
Content changes we get on MDN are related to a variety of work streams, for example:
- Day-to-day content improvement work — new APIs, new CSS properties, and other significant platform updates and content additions, usually done by MDN staff working for Mozilla, Google, Open Web Docs, Samsung, etc., but also sometimes by community volunteers.
- "Drive-by fixes" — small updates done to the site to fix typos, grammatical issues, and technical inaccuracies, usually as they are found by users of MDN.
- MDN content bug fixes, usually done by volunteers to close issues on this repo.
Regardless of how a content change is done, they will be submitted as pull requests on this repo, which will require rapid reviewing and merging to ensure that the site does not get out-of-date. This is being handled as follows:
- Different MDN staff members and volunteers have been assigned as "topic review owners", meaning that when a pull request comes in related to a particular topic area of the site (e.g. the CSS reference, or the learning area), it will be assigned to that area's topic review owner(s) and they will receive an e-mail notification asking for a review. This is being handled using a CODEOWNERS file, in which particular content directories are assigned to the topics review owner's GitHub usernames.
- Once the review has been done and the pull request has been approved, the reviewer should also merge the pull request.
- The site will be rebuilt once every 24 hours to ensure that the content does not get too stale.
If you are reviewing mdn content changes, read through the following guidelines. There's quite a lot here, but don't worry if you don't review perfectly in accordance with all of these points immediately. It is more important to make sure the content is readable, useful, correct, and not inappropriate, than it is to follow every guideline to the letter.
- Familiarize yourself with the MDN Code example guidelines and make sure that code examples follow the guidelines. You'll get used to them eventually, and we are intending to automatically lint against our guidelines at some point in the future.
- Familiarize yourself with the MDN Writing style guide, and use it to inform your reviews of new text content.
- Familiarize yourself with the MDN pull request guidelines.
The key points here are
- You have the right to request more information to help your review if the submitter has not explained why they are making this change.
- You have the right to close a pull request if it is too complex and/or contains multiple unrelated changes and ask the submitter to submit their changes in smaller atomic chunks.
- When reviewing a pull request, use the GitHub review tools. Use "Request changes" when submitting a review that will require the submitter to do some more work, or "Approve" if the submission is ready to add and you want to merge it. Reviewing proposed changes in a pull request is also useful if you want more information.
- Be polite and constructive at all times when writing review comments, or otherwise interacting with the submitter and other community members. We are all bound by our Code of Conduct when contributing to MDN, which means adhering to Mozilla's Community Participation Guidelines. If anyone has engaged in behavior that is potentially illegal or makes you or someone else feel unsafe, unwelcome, or uncomfortable, you are encouraged to report it. We want MDN to be a welcoming, friendly community that we can all be proud of.
- If a pull request is fine apart from a small typo or some other minor
issue, you might want to just fix the issue yourself rather than ask the
submitter to change it. You can do this provided the PR has been set up
to allow changes (see Allowing changes to a pull request branch created from a fork
for more details). If you are not sure how to make changes to someone
else's pull request, @vkWeb wrote some nice
simple instructions on how to do this on the command line; see
ReviewPRCommands.
- Alternatively, you can edit files using the GitHub UI — go to the pull request's "Files changed" tab, find the file you want to edit, and choose "three dot" menu (...) > Edit file.
- If a pull request is fine in itself, is a clear improvement to the content, and makes the change it claims to make in the description, you should as a rule merge it, even if you can see other improvements that could be made in the same file. If you want to ensure that these other improvements will be taken care of, file a follow-up issue or pull request of your own to address them.
- If you don't understand a content change that you've been selected to
review, or feel that it is too large and complex for you to deal with,
don't panic! Feel free to reach out to someone else to ask for help,
like a colleague, or someone else in your group of topic review owners
(if you know who they are). If you are not sure who to approach for help,
then ping our
@core-yari-contentgroup to ask for help. - Related to the above point, it is rare that you'll be required to review a large, complex content change with no warning, like a complete page rewrite, or the addition of several new reference pages or tutorials. Usually such changes are done as part of specific work streams where the content has been approved for addition, and reviewer(s) have been assigned already. In such cases, the PR should be linked to an issue that explains all these details. If you are not sure, ask the submitter if they need a review of the content, and where the rationale behind the change is explained. Ping our team on MDN Web Docs chat room to ask for help if you are still not sure, or if you think the content is suspicious.
Note: You may encounter merge conflicts as you review pull requests, if a another pull request that touches some of the same files got merged before the one you are reviewing. Addressing merge conflicts is a useful resource to help you. Feel free also to ask your team(s) for help if you need it.
Some of the pull requests submitted on the content repo relate to specific
workstreams being undertaken by browser vendors or other organizations that
have a defined set of writers and reviewers. In these cases, the submitter
of the PR will include the username of the reviewer in a line at the bottom
of the pull request description, for example:
reviewer: @jpmedley
Upon submitting the pull request, they will request a review from the reviewer specified in the pull request description. Once that reviewer has approved the new content, they will then ask you for an approval as required by the CODEOWNERS system for the pull request to be mergeable.
Therefore, if you receive a pull request review request and then see that you have been overridden with another reviewer in the manner described above, then don't review the pull request — just wait for an approval request.
The following specific topic areas are being reviewed by the kind souls listed underneath them. Be kind to them, and thank them for all the help they give to this project. If you would like to help with MDN content reviews, get in touch with us.
Note that changes to any content areas not explicitly listed below will be handled by the @core-yari-content team, which currently consists of @Rumyra.
- Web accessibility content:
- General learning content:
- CSS learning content:
- Server-side learning content:
- MDN meta docs — the @yari-content-mdn team, which consists of:
- Firefox Developer Tools content:
- Mozilla Add-ons reference content — the @yari-content-mozilla-add-ons team, which consists of:
- HTTP reference content — the @yari-content-http team, which consists of:
- CSS reference content — the @yari-content-css team, which consists of:
- HTML reference content — the @yari-content-html team, which consists of:
- JavaScript reference content — the @yari-content-javascript team, which consists of:
- Web API reference content — the @yari-content-web-api team, which consists of:
- SVG reference content — the @yari-content-svg team, which consists of:
The following folks used to be in one or more of our review teams, but no longer have the time to contribute; we want to give them our sincere thanks for all their help.