Lokasi ngalangkungan proxy:   [ UP ]  
[Ngawartoskeun bug]   [Panyetelan cookie]                
Skip to content

translate-transact-sql.md - Spelling, Grammar, Wording#1333

Merged
MightyPen merged 1 commit intoMicrosoftDocs:livefrom
LarnuUK:patch-7
Nov 27, 2018
Merged

translate-transact-sql.md - Spelling, Grammar, Wording#1333
MightyPen merged 1 commit intoMicrosoftDocs:livefrom
LarnuUK:patch-7

Conversation

@LarnuUK
Copy link
Copy Markdown
Contributor

@LarnuUK LarnuUK commented Nov 25, 2018

The document was quite poorly worded. It contained spelling errors, and read like broken English. It also contained statements that weren't true; if NULL is passed for any parameter, then NULL is returned, however the document states the opposite. The word "should" in the document implies that something isn't always the case. For example "he behavior of the TRANSLATE function should be identical to the REPLACE function. " If TRANSLATE doesn't always operate like REPLACE, when does it not? It's not that TRANSLATE should, it does. I've also removed the repetition of this statement.

The document was quite poorly worded. It contained spelling errors, and read like broken English. It also contained statements that weren't true; if NULL is passed for any parameter, then NULL is returned, however the document states the opposite. The word "should" in the document implies that something isn't always the case. For example "he behavior of the TRANSLATE function should be identical to the REPLACE function. " If TRANSLATE doesn't always operate like REPLACE, when does it not? It's not that TRANSLATE should, it does. I've also removed the repetition of this statement.
@PRMerger12
Copy link
Copy Markdown
Contributor

@LarnuUK : Thanks for your contribution! The author, @MashaMSFT, has been notified to review your proposed change.

@MightyPen
Copy link
Copy Markdown
Contributor

@MashaMSFT Hi Masha, Yes I do recommend that Microsoft accept these changes to TRANSLATE. These changes are nice improvements. I will Merge this public PR 1333 now.
.
CC: @LarnuUK Thank you Thom (LarnuUK), these are high quality improvements.
.
GeneMi , 2018/11/27 11:14am

@MightyPen MightyPen merged commit 271c37e into MicrosoftDocs:live Nov 27, 2018
@LarnuUK LarnuUK deleted the patch-7 branch November 27, 2018 19:29
@MightyPen
Copy link
Copy Markdown
Contributor

Minor FYI, private repo PR 8232 was also spurred by this public repo PR 1333. PR 8232 made the four-REPLACE code example much more readable to the human eye.

@LarnuUK
Copy link
Copy Markdown
Contributor Author

LarnuUK commented Nov 28, 2018

Was this supposed to be live today @MightyPen ? I can see the new formatting for REPLACE, however, the typoes, poor grammar and incorrect information is still there.

@MightyPen
Copy link
Copy Markdown
Contributor

@LarnuUK No, not yet. We cannot promise much about how long it takes our system to make live changes from Pull Requests such as this. Perhaps a "public PR" such as this PR 1333 are visible on the live Docs website within 48 hours; but there are several reasons why this or that PR, or on this or that day, a public PR might not be visible live for several days. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants