Skip to content

Fix grammatical typos - #4853

Merged
mstfbl merged 5 commits into
dotnet:masterfrom
mstfbl:typoCorrections
Feb 19, 2020
Merged

Fix grammatical typos#4853
mstfbl merged 5 commits into
dotnet:masterfrom
mstfbl:typoCorrections

Conversation

@mstfbl

@mstfbl mstfbl commented Feb 18, 2020

Copy link
Copy Markdown
Contributor

Fix grammatical typos. Utilized Visual Studio Spell Checker to quickly find and implement typo fixes.

@mstfbl
mstfbl requested a review from a team as a code owner February 18, 2020 23:11
@mstfbl mstfbl changed the title Fix grammatical typo Fix grammatical typos Feb 18, 2020

@gvashishtha gvashishtha left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is there a way to ensure that these typo checks run as part of CI builds?

Comment thread docs/samples/Microsoft.ML.Samples/Dynamic/TensorFlow/TextClassification.cs Outdated
@mstfbl

mstfbl commented Feb 18, 2020

Copy link
Copy Markdown
Contributor Author

@gvashishtha Visual Studio Spell Checker actually marked certain correctly spelled words that are not in its dictionary as misspelled, so the human factor here is necessary to ascertain when the Spell Checker is correct or not. As such I don't think it would be realistically viable to have this as part of the CI build check.

@codemzs

codemzs commented Feb 18, 2020

Copy link
Copy Markdown
Member

@gvashishtha The point of this PR is to fix typos(see the title) not enable tests as part of CI build so blocking PR on it for that reason is not acceptable. @mstfbl feel free to dismiss the block.

@mstfbl
mstfbl dismissed gvashishtha’s stale review February 18, 2020 23:55

Out of PR's scope

@mstfbl
mstfbl merged commit 140cb70 into dotnet:master Feb 19, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Mar 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants