Skip to content
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-39569 changed limited to unsupported for setParameter #6275

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

sonderdonk-mdb
Copy link
Contributor

DESCRIPTION

changed "this command has limited support" to "this command is unsupported" for setParameter compatibility section

STAGING

https://preview-mongodbsonderdonkmdb.gatsbyjs.io/docs/DOCSP-39569/reference/command/setParameter/#compatibility

JIRA

https://jira.mongodb.org/browse/DOCSP-39569

BUILD LOG

https://workerpool-boxgs.mongodbstitch.com/pages/job.html?collName=queue&jobId=666affc2ebce19f3148c3a21

SELF-REVIEW CHECKLIST

  • Does each file have 3-5 taxonomy facet tags?
    See the taxonomy tagging instructions and this example PR
  • Is this free of any warnings or errors in the RST?
  • Is this free of spelling errors?
  • Is this free of grammatical errors?
  • Is this free of staging / rendering issues?
  • Are all the links working?

EXTERNAL REVIEW REQUIREMENTS

What's expected of an external reviewer?

@kennethdyer kennethdyer self-requested a review June 13, 2024 16:09
Copy link
Contributor

@kennethdyer kennethdyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this work @sonderdonk-mdb. Left a small nonblocking suggestion for your consideration. LGTM.

@@ -1,4 +1,4 @@
.. note::
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nonblocking Suggestion:

This is not in the scope of your project, but if you apply a two space indent to the include directive used to call this RST file, it'll render as a child of the lists item instead of breaking the list into two separate lists.

Bonus: Since the robots work from simplified HTML, I suspect that'll help their comprehension.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

spoke to Kenneth over slack, ended up not implementing

@sonderdonk-mdb
Copy link
Contributor Author

Will Pierce gave external review and approved

@sarah-olson-mongodb sarah-olson-mongodb merged commit 5e714b0 into mongodb:master Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants