Skip to content
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

Add prepared logs to the logging test #583

Merged
merged 6 commits into from
Jun 13, 2024

Update changelog

ecd03b7
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

Add prepared logs to the logging test #583

Update changelog
ecd03b7
Select commit
Loading
Failed to load commit list.
GitHub Actions / LanguageTool succeeded Jun 12, 2024 in 1s

reviewdog [LanguageTool] report

reported by reviewdog 🐶

Findings (0)
Filtered Findings (15)

.github/pull_request_template.md|16 col 36| This expression is normally spelled as one or with a hyphen. (EN_COMPOUNDS_NON_TRIVIAL)
Suggestions: non-trivial, nontrivial
Rule: https://community.languagetool.org/rule/show/EN_COMPOUNDS_NON_TRIVIAL?lang=en-US
Category: MISC
CHANGELOG.md|172 col 17| This expression is normally spelled as one or with a hyphen. (EN_COMPOUNDS_SIDE_CAR)
Suggestions: side-car, sidecar
Rule: https://community.languagetool.org/rule/show/EN_COMPOUNDS_SIDE_CAR?lang=en-US
Category: MISC
README.md|60 col 19| It appears that hyphens are missing in the adjective “up-to-date”. (UP_TO_DATE_HYPHEN[2])
Suggestions: up-to-date
URL: https://languagetool.org/insights/post/hyphen/#hyphenated-phrases-with-more-than-one-hyphen
Rule: https://community.languagetool.org/rule/show/UP_TO_DATE_HYPHEN?lang=en-US&subId=2
Category: COMPOUNDING
docs/modules/opa/pages/getting_started/first_steps.adoc|72 col 1| In American English, “take a look” is more commonly used. (HAVE_A_LOOK[5])
Suggestions: Take a look
Rule: https://community.languagetool.org/rule/show/HAVE_A_LOOK?lang=en-US&subId=5
Category: BRITISH_ENGLISH
docs/modules/opa/pages/implementation-notes.adoc|11 col 27| 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
docs/modules/opa/pages/index.adoc|5 col 1| 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
docs/modules/opa/pages/index.adoc|12 col 3| 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
docs/modules/opa/pages/index.adoc|48 col 1| A comma may be missing after the conjunctive/linking adverb ‘Currently’. (SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA[1])
Suggestions: Currently,
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
docs/modules/opa/pages/index.adoc|65 col 7| 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
docs/modules/opa/pages/usage-guide/user-info-fetcher.adoc|50 col 1| A comma may be missing after the conjunctive/linking adverb ‘Currently’. (SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA[1])
Suggestions: Currently,
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
docs/modules/opa/pages/usage-guide/user-info-fetcher.adoc|57 col 3| It appears that a hyphen is missing in the noun “to-do” (= task) or did you mean the verb “to do”? (TO_DO_HYPHEN[3])
Suggestions: to-do, to do
URL: https://languagetool.org/insights/post/hyphen/#compound-adjectives-with-hyphens
Rule: https://community.languagetool.org/rule/show/TO_DO_HYPHEN?lang=en-US&subId=3
Category: GRAMMAR
docs/modules/opa/pages/usage-guide/user-info-fetcher.adoc|70 col 93| This phrase is redundant. Consider using “outside”. (OUTSIDE_OF[1])
Suggestions: outside
URL: https://languagetool.org/insights/post/wordiness/
Rule: https://community.languagetool.org/rule/show/OUTSIDE_OF?lang=en-US&subId=1
Category: REDUNDANCY
docs/modules/opa/pages/usage-guide/user-info-fetcher.adoc|101 col 14| 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
docs/modules/opa/pages/usage-guide/user-info-fetcher.adoc|109 col 15| The word “lookup” is a noun. The verb is spelled with a space. (NOUN_VERB_CONFUSION[1])
Suggestions: look up
Rule: https://community.languagetool.org/rule/show/NOUN_VERB_CONFUSION?lang=en-US&subId=1
Category: GRAMMAR
tests/README-templating.md|63 col 101| Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE[1])
Suggestions: , and
URL: https://languagetool.org/insights/post/types-of-sentences/#compound-sentence
Rule: https://community.languagetool.org/rule/show/COMMA_COMPOUND_SENTENCE?lang=en-US&subId=1
Category: PUNCTUATION