Skip to content

Fix some typos#7665

Merged
KevinRansom merged 1 commit into
dotnet:masterfrom
auduchinok:spelling
Oct 30, 2019
Merged

Fix some typos#7665
KevinRansom merged 1 commit into
dotnet:masterfrom
auduchinok:spelling

Conversation

@auduchinok

@auduchinok auduchinok commented Sep 28, 2019

Copy link
Copy Markdown
Member

I'm trying to make the spell checker a bit less unhappy about FSharp.Compiler.Service.sln so the scroll bar shows less typo warnings.

This PR fixes some typos across the FCS project and adds a team-shared ReSharper dictionary with commonly used abbreviations and some words the built-in dictionary doesn't have.

@auduchinok

Copy link
Copy Markdown
Member Author

This is ready.

@cartermp cartermp requested review from KevinRansom and dsyme October 5, 2019 16:47

@cartermp cartermp 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.

I think this is a good one to take. Always irksome to see the typos whenever I come across them.

Just for reference for others at MS, the .DotSettings file is something that JetBrains tooling can pick up and work with. We don't have an equivalent spellchecker that's language-agnostic in VS.

@KevinRansom KevinRansom 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.

Impressively thorough.

Thanks for this

Kevin

@KevinRansom KevinRansom merged commit 9be86c2 into dotnet:master Oct 30, 2019
nosami pushed a commit to xamarin/visualfsharp that referenced this pull request Feb 23, 2021
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.

3 participants