-
Notifications
You must be signed in to change notification settings - Fork 34
Docsp 13857 #16
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
Docsp 13857 #16
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.
LGTM with some nits and suggestions!
@kevinAlbs Ready for dbx 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.
LGTM minus a few suggestions!
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.
👍
Pull Request Info
This includes a few key changes along with implementing the page. It:
snooty.toml
to help with versions in links.uri
declaration leveraging environmental variablesgo.mod
and ago.sum
file to enable running files in thecode-snippets
directly in place (no copy/paste from some other project).Issue JIRA link:
https://jira.mongodb.org/browse/DOCSP-13857
Docs staging link (requires sign-in on MongoDB Corp SSO):
https://docs-mongodbcom-staging.corp.mongodb.com/golang/docsworker-xlarge/DOCSP-13857/usage-examples/insertOne/
Self-Review Checklist