Skip to content

incorrect grammar fixes - #10851

Merged
Jiashuo Li (jiasli) merged 5 commits into
Azure:devfrom
kishdubey:dev
Oct 16, 2019
Merged

Jiashuo Li (jiasli) merged 5 commits into
Azure:devfrom
kishdubey:dev

Conversation

@kishdubey

@kishdubey Kish Dubey (kishdubey) commented Oct 16, 2019

Copy link
Copy Markdown
Contributor

Addresses issue #10849. Initially the warning message's grammar was fixed, but after further discussion within the comments the entire message was removed. Also addresses issue ##10848 for incorrect grammar.


This checklist is used to make sure that common guidelines for a pull request are followed.

  • The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

  • I adhere to the Command Guidelines.

@msftclas

Microsoft Contribution License Agreements (msftclas) commented Oct 16, 2019

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@kishdubey Kish Dubey (kishdubey) changed the title incorrect English on warning message removed incorrect grammar fixes Oct 16, 2019
@jiasli Jiashuo Li (jiasli) self-assigned this Oct 16, 2019
@Juliehzl

Copy link
Copy Markdown
Contributor

LGTM

@jiasli
Jiashuo Li (jiasli) merged commit c375ee5 into Azure:dev Oct 16, 2019
@jiasli

Copy link
Copy Markdown
Contributor

Kish Dubey (@kishdubey), thanks a lot for the contribution.

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.

4 participants