-
Notifications
You must be signed in to change notification settings - Fork 52
Release 4 4 #287
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
Release 4 4 #287
Conversation
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. I think this may also need to update the versions in the redirects file as well (can't comment as the file has no changes in this PR)
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.
LGTM. One of the comments from last round about passive voice and future tense was not addressed. Mentioning it again as I think you agreed with the note (no response).
Pull Request Info
This includes the changes necessary for a 4.4 branch to be cut. I had to make changes to the typescript limitations page name since it errored upon updating the version and some content was incoherent.
Issue JIRA link:
N/A
Snooty build log:
Paste your workerpool job link here
Docs staging link (requires sign-in on MongoDB Corp SSO):
https://docs-mongodbcom-staging.corp.mongodb.com/node/docsworker-xlarge/release-4-4/whats-new/
Self-Review Checklist
If your page documents a concept, does it meet the following criteria?