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-27679:Make placeholder text consistent #108

Merged

Conversation

nickldp
Copy link
Contributor

@nickldp nickldp commented Jul 18, 2023

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-27679
Staging - https://docs-mongodbcom-staging.corp.mongodb.com/csharp/docsworker-xlarge/DOCSP-27679-consistent-placeholder-text/

Self-Review Checklist

  • Is this free of any warnings or errors in the RST?
  • Did you run a spell-check?
  • Did you run a grammar-check?
  • Are all the links working?

source/faq.txt Outdated
@@ -97,7 +97,7 @@ applications:

.. code-block:: csharp

var client = new MongoClient("<connection string>");
Copy link
Collaborator

Choose a reason for hiding this comment

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

S: I think <connection string> is the best placeholder, suggesting that you change the placeholder to this in all locations.

Comment on lines 65 to 66
var settings = MongoClientSettings.FromConnectionString("<your
MongoDB URI>");
Copy link
Collaborator

Choose a reason for hiding this comment

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

I: make sure you revert the line breaks that your formatter creates when it comes to code examples.

Suggested change
var settings = MongoClientSettings.FromConnectionString("<your
MongoDB URI>");
var settings = MongoClientSettings.FromConnectionString("<your MongoDB URI>");

@nickldp nickldp requested a review from rustagir July 18, 2023 19:30
Copy link
Collaborator

@rustagir rustagir left a comment

Choose a reason for hiding this comment

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

lgtm!

@nickldp nickldp merged commit 23d1593 into mongodb:master Jul 19, 2023
1 check passed
nickldp added a commit that referenced this pull request Jul 19, 2023
* DOCSP-27679:Make placeholder text consistent

* changing placeholder

* fixed line wraps

(cherry picked from commit 23d1593)
nickldp added a commit that referenced this pull request Jul 19, 2023
* DOCSP-27679:Make placeholder text consistent

* changing placeholder

* fixed line wraps

(cherry picked from commit 23d1593)
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.

2 participants