-
Notifications
You must be signed in to change notification settings - Fork 4
Revert "Move config (#125)" #126
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
LanguageTool
[LanguageTool] reported by reviewdog 🐶
Don’t put a space before the full stop. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: .
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
* .editorconfig |
[LanguageTool] reported by reviewdog 🐶
Don’t put a space before the full stop. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: .
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
* .globalconfig |
[LanguageTool] reported by reviewdog 🐶
The official name of this software platform is spelled with a capital “H”. (GITHUB[1])
Suggestions: GitHub
URL: https://en.wikipedia.org/wiki/GitHub
Rule: https://community.languagetool.org/rule/show/GITHUB?lang=en-US&subId=1
Category: CASING
1. [gist.github.com](https://gist.github.com/jonlabelle/34993ee032c26420a0943b1c9d106cdc) |
[LanguageTool] reported by reviewdog 🐶
Don’t put a space after the opening parenthesis. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: [
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
<!CDATA[[ |
[LanguageTool] reported by reviewdog 🐶
Don’t put a space before the closing parenthesis. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: ]
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
- Proj1001: Use analyzers for packages. (NEW RULE) |
[LanguageTool] reported by reviewdog 🐶
Did you mean “had liked”? (IF_WOULD_HAVE_VBN[1])
Suggestions: had liked
Rule: https://community.languagetool.org/rule/show/IF_WOULD_HAVE_VBN?lang=en-US&subId=1
Category: GRAMMAR
even if you would have liked them to. |
[LanguageTool] reported by reviewdog 🐶
A comma may be missing after the conjunctive/linking adverb ‘Therefore’. (SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA[1])
Suggestions: Therefore,
URL: https://languagetool.org/insights/post/linking-words/
Rule: https://community.languagetool.org/rule/show/SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA?lang=en-US&subId=1
Category: PUNCTUATION
They are used as a default value for the `Version` node. |
[LanguageTool] reported by reviewdog 🐶
Did you mean: “By default,”? (BY_DEFAULT_COMMA[1])
Suggestions: By default,
Rule: https://community.languagetool.org/rule/show/BY_DEFAULT_COMMA?lang=en-US&subId=1
Category: PUNCTUATION
[LanguageTool] reported by reviewdog 🐶
The official name of this software platform is spelled with a capital “H”. (GITHUB[1])
Suggestions: GitHub
URL: https://en.wikipedia.org/wiki/GitHub
Rule: https://community.languagetool.org/rule/show/GITHUB?lang=en-US&subId=1
Category: CASING
2. [gist.github.com](https://gist.github.com/jonlabelle/34993ee032c26420a0943b1c9d106cdc) |
[LanguageTool] reported by reviewdog 🐶
Did you mean: “By default,”? (BY_DEFAULT_COMMA[1])
Suggestions: By default,
Rule: https://community.languagetool.org/rule/show/BY_DEFAULT_COMMA?lang=en-US&subId=1
Category: PUNCTUATION
They are used as a default value for the `Version` node. |
[LanguageTool] reported by reviewdog 🐶
A comma may be missing after the conjunctive/linking adverb ‘Therefore’. (SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA[1])
Suggestions: Therefore,
URL: https://languagetool.org/insights/post/linking-words/
Rule: https://community.languagetool.org/rule/show/SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA?lang=en-US&subId=1
Category: PUNCTUATION
[LanguageTool] reported by reviewdog 🐶
The official name of this software platform is spelled with a capital “H”. (GITHUB[1])
Suggestions: GitHub
URL: https://en.wikipedia.org/wiki/GitHub
Rule: https://community.languagetool.org/rule/show/GITHUB?lang=en-US&subId=1
Category: CASING
1. [gist.github.com](https://gist.github.com/jonlabelle/34993ee032c26420a0943b1c9d106cdc) |
[LanguageTool] reported by reviewdog 🐶
The verb ‘helps’ is used with an infinitive. (ADVISE_VBG[3])
Suggestions: to prevent
, prevent
URL: https://learningenglish.voanews.com/a/everyday-grammar-gerunds-infinitives/2722827.html
Rule: https://community.languagetool.org/rule/show/ADVISE_VBG?lang=en-US&subId=3
Category: GRAMMAR
This helps preventing any unintentional API changes. |
[LanguageTool] reported by reviewdog 🐶
The word “opt-in” is a noun. The verb is spelled with a white space. (NOUN_VERB_CONFUSION[6])
Suggestions: opt in
Rule: https://community.languagetool.org/rule/show/NOUN_VERB_CONFUSION?lang=en-US&subId=6
Category: GRAMMAR
is enabled, it is advised to opt-in to the strict attribute compatibility checks. |
[LanguageTool] reported by reviewdog 🐶
The word “opt-in” is a noun. The verb is spelled with a white space. (NOUN_VERB_CONFUSION[6])
Suggestions: opt in
Rule: https://community.languagetool.org/rule/show/NOUN_VERB_CONFUSION?lang=en-US&subId=6
Category: GRAMMAR
is enabled, it is advised to opt-in to the strict parameter name compatibility checks. |
[LanguageTool] reported by reviewdog 🐶
Did you mean “setting”? Or maybe you should add a pronoun? In active voice, ‘advise’ + ‘to’ takes an object, usually a pronoun. (ALLOW_TO[1])
Suggestions: setting
URL: http://www.ef.com/english-resources/english-grammar/gerund-equals-infinitive/
Rule: https://community.languagetool.org/rule/show/ALLOW_TO?lang=en-US&subId=1
Category: GRAMMAR
rule will advise to set `<IsPublishable>` to `false`. |
[LanguageTool] reported by reviewdog 🐶
A comma may be missing after the conjunctive/linking adverb ‘Hence’. (SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA[1])
Suggestions: Hence,
URL: https://languagetool.org/insights/post/linking-words/
Rule: https://community.languagetool.org/rule/show/SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA?lang=en-US&subId=1
Category: PUNCTUATION
Test projects should only be responsible for running tests. Hence if they do |
[LanguageTool] reported by reviewdog 🐶
A comma may be missing after the conjunctive/linking adverb ‘Hence’. (SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA[1])
Suggestions: Hence,
URL: https://languagetool.org/insights/post/linking-words/
Rule: https://community.languagetool.org/rule/show/SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA?lang=en-US&subId=1
Category: PUNCTUATION
Test projects should only be responsible for running tests. Hence if they |
[LanguageTool] reported by reviewdog 🐶
This word is normally spelled with a hyphen. (EN_COMPOUNDS_HUMAN_READABLE)
Suggestions: human-readable
Rule: https://community.languagetool.org/rule/show/EN_COMPOUNDS_HUMAN_READABLE?lang=en-US
Category: MISC
within a single `<ItemGroup>` in order to make it more |
[LanguageTool] reported by reviewdog 🐶
Did you mean “enabling”? Or maybe you should add a pronoun? In active voice, ‘advise’ + ‘to’ takes an object, usually a pronoun. (ALLOW_TO[1])
Suggestions: enabling
URL: http://www.ef.com/english-resources/english-grammar/gerund-equals-infinitive/
Rule: https://community.languagetool.org/rule/show/ALLOW_TO?lang=en-US&subId=1
Category: GRAMMAR
Hence, this rules advises to enable Central Package Management by |
[LanguageTool] reported by reviewdog 🐶
A comma may be missing after the conjunctive/linking adverb ‘Otherwise’. (SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA[1])
Suggestions: Otherwise,
URL: https://languagetool.org/insights/post/linking-words/
Rule: https://community.languagetool.org/rule/show/SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA?lang=en-US&subId=1
Category: PUNCTUATION
`Directory.Packages.props` or a shared `.props` file. Otherwise it would defy |
[LanguageTool] reported by reviewdog 🐶
Use “a” instead of ‘an’ if the following word doesn’t start with a vowel sound, e.g. ‘a sentence’, ‘a university’. (EN_A_VS_AN)
Suggestions: a
URL: https://languagetool.org/insights/post/indefinite-articles/
Rule: https://community.languagetool.org/rule/show/EN_A_VS_AN?lang=en-US
Category: MISC
likely an mistake; or `Version` was intended, or the CPM has been unintentionally |
[LanguageTool] reported by reviewdog 🐶
Use “a” instead of ‘an’ if the following word doesn’t start with a vowel sound, e.g. ‘a sentence’, ‘a university’. (EN_A_VS_AN)
Suggestions: a
URL: https://languagetool.org/insights/post/indefinite-articles/
Rule: https://community.languagetool.org/rule/show/EN_A_VS_AN?lang=en-US
Category: MISC
likely an mistake; or `VersionOverride` was intended, or the CPM has been |
[LanguageTool] reported by reviewdog 🐶
This sentence does not start with an uppercase letter. (UPPERCASE_SENTENCE_START)
Suggestions: Or
URL: https://languagetool.org/insights/post/spelling-capital-letters/
Rule: https://community.languagetool.org/rule/show/UPPERCASE_SENTENCE_START?lang=en-US
Category: CASING
[LanguageTool] reported by reviewdog 🐶
Consider using either the past participle “detected” or the present participle “detecting” here. (BEEN_PART_AGREEMENT[9])
Suggestions: detected
, detecting
URL: https://languagetool.org/insights/post/verb-tenses/
Rule: https://community.languagetool.org/rule/show/BEEN_PART_AGREEMENT?lang=en-US&subId=9
Category: GRAMMAR
The purpose of this rule is detect the absence of the .NET project file analyzers. |
[LanguageTool] reported by reviewdog 🐶
Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym. (ENGLISH_WORD_REPEAT_BEGINNING_RULE)
Rule: https://community.languagetool.org/rule/show/ENGLISH_WORD_REPEAT_BEGINNING_RULE?lang=en-US
Category: REPETITIONS_STYLE
| [Libplanet.Analyzers](https://www.nuget.org/packages/Libplanet.Analyzers) | [Libplanet](https://www.nuget.org/packages?q=Libplanet) | Any | |
[LanguageTool] reported by reviewdog 🐶
Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym. (ENGLISH_WORD_REPEAT_BEGINNING_RULE)
Rule: https://community.languagetool.org/rule/show/ENGLISH_WORD_REPEAT_BEGINNING_RULE?lang=en-US
Category: REPETITIONS_STYLE
| [NSubstitute.Analyzers.CSharp](https://www.nuget.org/packages/NSubstitute.Analyzers.CSharp) | [NSubstitute](https://www.nuget.org/packages?q=NSubstitute) | C# | |
[LanguageTool] reported by reviewdog 🐶
Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym. (ENGLISH_WORD_REPEAT_BEGINNING_RULE)
Rule: https://community.languagetool.org/rule/show/ENGLISH_WORD_REPEAT_BEGINNING_RULE?lang=en-US
Category: REPETITIONS_STYLE
| [ZeroFormatter.Analyzer](https://www.nuget.org/packages/ZeroFormatter.Analyzer) | [ZeroFormatter](https://www.nuget.org/packages?q=ZeroFormatter) | Any | |
[LanguageTool] reported by reviewdog 🐶
If ‘type’ is a classification term, ‘a’ is not necessary. Use “type of”. (The phrases ‘kind of’ and ‘sort of’ are informal if they mean ‘to some extent’.) (KIND_OF_A[1])
Suggestions: type of
Rule: https://community.languagetool.org/rule/show/KIND_OF_A?lang=en-US&subId=1
Category: GRAMMAR
As a result the output type of a TUnit test project must be an executable. |
[LanguageTool] reported by reviewdog 🐶
Two consecutive dots (DOUBLE_PUNCTUATION)
Suggestions: .
, …
URL: https://languagetool.org/insights/post/punctuation-guide/#what-are-periods
Rule: https://community.languagetool.org/rule/show/DOUBLE_PUNCTUATION?lang=en-US
Category: PUNCTUATION
> machine-readable.([..])[https://en.wikipedia.org/wiki/XML] |
[LanguageTool] reported by reviewdog 🐶
Unpaired symbol: ‘]’ seems to be missing (EN_UNPAIRED_BRACKETS)
URL: https://languagetool.org/insights/post/punctuation-guide/#what-are-parentheses
Rule: https://community.languagetool.org/rule/show/EN_UNPAIRED_BRACKETS?lang=en-US
Category: PUNCTUATION
# Proj1701: Use <![CDATA[ for large texts |
[LanguageTool] reported by reviewdog 🐶
Don’t put a space after the opening parenthesis. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: [
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
# Proj1701: Use <![CDATA[ for large texts |
[LanguageTool] reported by reviewdog 🐶
Don’t put a space after the opening parenthesis. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: [
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
<![CDATA[ |
[LanguageTool] reported by reviewdog 🐶
Don’t put a space before the closing parenthesis. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: ]
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
- Proj1001: Use analyzers for packages. (NEW RULE) |
[LanguageTool] reported by reviewdog 🐶
Two consecutive dots (DOUBLE_PUNCTUATION)
Suggestions: .
, …
URL: https://languagetool.org/insights/post/punctuation-guide/#what-are-periods
Rule: https://community.languagetool.org/rule/show/DOUBLE_PUNCTUATION?lang=en-US
Category: PUNCTUATION
> machine-readable.([..])[https://en.wikipedia.org/wiki/XML] |
[LanguageTool] reported by reviewdog 🐶
Use “a” instead of ‘an’ if the following word doesn’t start with a vowel sound, e.g. ‘a sentence’, ‘a university’. (EN_A_VS_AN)
Suggestions: a
URL: https://languagetool.org/insights/post/indefinite-articles/
Rule: https://community.languagetool.org/rule/show/EN_A_VS_AN?lang=en-US
Category: MISC
Therefor, this rule will warn when a file is stored with an UTF-8 BOM. |
[LanguageTool] reported by reviewdog 🐶
Unpaired symbol: ‘]’ seems to be missing (EN_UNPAIRED_BRACKETS)
URL: https://languagetool.org/insights/post/punctuation-guide/#what-are-parentheses
Rule: https://community.languagetool.org/rule/show/EN_UNPAIRED_BRACKETS?lang=en-US
Category: PUNCTUATION
[Header2]] |
[LanguageTool] reported by reviewdog 🐶
Put a space after the comma. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: , json
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
[*.{cs,json,cshtml,ts}] |
[LanguageTool] reported by reviewdog 🐶
Put a space after the comma. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: , ts
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
[*.{cs,json,cshtml,ts}] |
[LanguageTool] reported by reviewdog 🐶
Put a space after the comma. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: , props
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
[*.{csproj,props,targets,config,nuspec}] |
[LanguageTool] reported by reviewdog 🐶
Put a space after the comma. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: , config
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
[*.{csproj,props,targets,config,nuspec}] |
[LanguageTool] reported by reviewdog 🐶
Put a space after the comma. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: , json
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
[*.{cs,json,cshtml,ts}] |
[LanguageTool] reported by reviewdog 🐶
Put a space after the comma. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: , ts
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
[*.{cs,json,cshtml,ts}] |
[LanguageTool] reported by reviewdog 🐶
Put a space after the comma. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: , props
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
[*.{csproj,props,targets,config,nuspec}] |
[LanguageTool] reported by reviewdog 🐶
Put a space after the comma. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: , config
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
[*.{csproj,props,targets,config,nuspec}] |
[LanguageTool] reported by reviewdog 🐶
Put a space after the comma. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: , json
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
[*.{cs,json}}] |
[LanguageTool] reported by reviewdog 🐶
Unpaired symbol: ‘{’ seems to be missing (EN_UNPAIRED_BRACKETS)
URL: https://languagetool.org/insights/post/punctuation-guide/#what-are-parentheses
Rule: https://community.languagetool.org/rule/show/EN_UNPAIRED_BRACKETS?lang=en-US
Category: PUNCTUATION
[*.{cs,json}}] |
[LanguageTool] reported by reviewdog 🐶
Put a space after the comma. (COMMA_PARENTHESIS_WHITESPACE)
Suggestions: , json
Rule: https://community.languagetool.org/rule/show/COMMA_PARENTHESIS_WHITESPACE?lang=en-US
Category: TYPOGRAPHY
[*.{cs,json}] |
|
||
``` ini | ||
is_global = true | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[LanguageTool] reported by reviewdog 🐶
This sentence does not start with an uppercase letter. (UPPERCASE_SENTENCE_START)
Suggestions: Dotnet
URL: https://languagetool.org/insights/post/spelling-capital-letters/
Rule: https://community.languagetool.org/rule/show/UPPERCASE_SENTENCE_START?lang=en-US
Category: CASING
``` ini | ||
is_global = true | ||
|
||
dotnet_diagnostic.Proj0010.severity = none # Define the <OutputType> node explicitly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[LanguageTool] reported by reviewdog 🐶
This sentence does not start with an uppercase letter. (UPPERCASE_SENTENCE_START)
Suggestions: Dotnet
URL: https://languagetool.org/insights/post/spelling-capital-letters/
Rule: https://community.languagetool.org/rule/show/UPPERCASE_SENTENCE_START?lang=en-US
Category: CASING
|
||
``` ini | ||
is_global = false | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[LanguageTool] reported by reviewdog 🐶
This sentence does not start with an uppercase letter. (UPPERCASE_SENTENCE_START)
Suggestions: Dotnet
URL: https://languagetool.org/insights/post/spelling-capital-letters/
Rule: https://community.languagetool.org/rule/show/UPPERCASE_SENTENCE_START?lang=en-US
Category: CASING
## Suppress specific warnings | ||
Adopted from C-style languages, it is possible to suppress | ||
individual violations and/or [false positives](https://en.wikipedia.org/wiki/False_positives_and_false_negatives). | ||
In a MS Build project file this would look like: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[LanguageTool] reported by reviewdog 🐶
Use “an” instead of ‘a’ if the following word starts with a vowel sound, e.g. ‘an article’, ‘an hour’. (EN_A_VS_AN)
Suggestions: an
URL: https://languagetool.org/insights/post/indefinite-articles/
Rule: https://community.languagetool.org/rule/show/EN_A_VS_AN?lang=en-US
Category: MISC
| [Proj0211](../rules/Proj0211.md) | Do not use the deprecated PackageLicenseUrl to include a license. | ||
| [Proj0212](../rules/Proj0212.md) | Consider including an icon with your package to help to visually differentiate it. It's a relatively small addition that can improve perception of package quality. | ||
| [Proj0213](../rules/Proj0213.md) | Consider adding the icon url for maximum compatibility. | ||
| [Proj0214](../rules/Proj0214.md) | Do include the package id. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[LanguageTool] reported by reviewdog 🐶
This abbreviation for “identification” is spelled all-uppercase. (ID_CASING[2])
Suggestions: ID
Rule: https://community.languagetool.org/rule/show/ID_CASING?lang=en-US&subId=2
Category: CASING
--- | ||
|
||
# Proj0031: Adopt preferred casing of nodes | ||
MS Build is (mostly) case insensitive. For project files, that means that the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[LanguageTool] reported by reviewdog 🐶
This word is normally spelled with a hyphen. (EN_COMPOUNDS_CASE_INSENSITIVE)
Suggestions: case-insensitive
Rule: https://community.languagetool.org/rule/show/EN_COMPOUNDS_CASE_INSENSITIVE?lang=en-US
Category: MISC
# Proj0031: Adopt preferred casing of nodes | ||
MS Build is (mostly) case insensitive. For project files, that means that the | ||
root node (`<Project>`) and its child nodes (such as `<PropertyGroup>` and | ||
`<ItemGroup>`) are case sensitive, but their ancestors are not. To prevent |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[LanguageTool] reported by reviewdog 🐶
This word is normally spelled with a hyphen. (EN_COMPOUNDS_CASE_SENSITIVE)
Suggestions: case-sensitive
Rule: https://community.languagetool.org/rule/show/EN_COMPOUNDS_CASE_SENSITIVE?lang=en-US
Category: MISC
# Proj0040: Run NuGet security audits on transitive dependencies too | ||
|
||
When [enabled](Proj0004.md), GitHub's vulnerability database is consulted to | ||
check for security issues that come with using any of the referenced packages. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[LanguageTool] reported by reviewdog 🐶
Did you mean: “By default,”? (BY_DEFAULT_COMMA[1])
Suggestions: By default,
Rule: https://community.languagetool.org/rule/show/BY_DEFAULT_COMMA?lang=en-US&subId=1
Category: PUNCTUATION
When [enabled](Proj0004.md), GitHub's vulnerability database is consulted to | ||
check for security issues that come with using any of the referenced packages. | ||
By default (this will change for .NET 10 and up) only direct dependencies are | ||
taken into consideration. This rule advises to include transitive dependencies too by setting |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[LanguageTool] reported by reviewdog 🐶
Did you mean “including”? Or maybe you should add a pronoun? In active voice, ‘advise’ + ‘to’ takes an object, usually a pronoun. (ALLOW_TO[1])
Suggestions: including
URL: http://www.ef.com/english-resources/english-grammar/gerund-equals-infinitive/
Rule: https://community.languagetool.org/rule/show/ALLOW_TO?lang=en-US&subId=1
Category: GRAMMAR
- [https://devblogs.microsoft.com/dotnet/enable-repeatable-package-restores-using-a-lock-file/](https://devblogs.microsoft.com/dotnet/enable-repeatable-package-restores-using-a-lock-file/) | ||
- [https://www.meziantou.net/faster-and-safer-nuget-restore-using-source-mapping-and-lock-files.htm](https://www.meziantou.net/faster-and-safer-nuget-restore-using-source-mapping-and-lock-files.htm) | ||
|
||
## When to disable this rule |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[LanguageTool] reported by reviewdog 🐶
This phrase is duplicated. You should probably use “this rule” only once. (PHRASE_REPETITION[1])
Suggestions: this rule
Rule: https://community.languagetool.org/rule/show/PHRASE_REPETITION?lang=en-US&subId=1
Category: GRAMMAR
This reverts commit 97780ca.