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

Typo correction and additional warning#5358

Merged
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
way0utwest:patch-31
Aug 24, 2020
Merged

Typo correction and additional warning#5358
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
way0utwest:patch-31

Conversation

@way0utwest
Copy link
Copy Markdown
Contributor

The original post had "can not" instead of cannot, which is a spelling error. This page also does not mention transactions, which is an important concept for variables.

The original post had "can not" instead of cannot, which is a spelling error. This page also does not mention transactions, which is an important concept for variables.
@PRMerger7
Copy link
Copy Markdown
Contributor

@way0utwest : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver
Copy link
Copy Markdown
Contributor

@rothja - Would you review the proposed changes? If the update is approved, I can change words to contractions where relevant per Microsoft style. Thanks.

@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Aug 24, 2020

This additional note is correct. @ktoliver So, we should be using a contraction "can't" instead of "cannot" per the style guide? Also, in the PR as it stands now, the wrong space was removed, but I think you see that "can notbe". Thanks.

@ktoliver
Copy link
Copy Markdown
Contributor

@rothja I'd suggest this:

> [!NOTE]
> - Variables can't be used in views. [Or, rewrite to avoid passive voice: "You can't use variables in a view" or "Views can't contain variables"]
> - Changes to variables aren't affected by the rollback of a transaction.

@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Aug 24, 2020

@ktoliver That works. Can you make the changes and sign off? Thanks.

@ktoliver
Copy link
Copy Markdown
Contributor

@ktoliver That works. Can you make the changes and sign off? Thanks.

Done. Thanks!

@ktoliver ktoliver merged commit 5fcf7bf into MicrosoftDocs:live Aug 24, 2020
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.

5 participants