Applying the QE feedback for the Logging guide #2497
Triggered via pull request
September 13, 2023 10:24
Status
Success
Total duration
2m 43s
Artifacts
–
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Annotations
4 warnings
Linting with Vale:
docs/src/main/asciidoc/logging.adoc#L248
[vale] reported by reviewdog 🐶
[Quarkus.TermsWarnings] Consider using 'might (for possiblity)' or 'can (for ability)' rather than 'may' unless updating existing content that uses the term.
Raw Output:
{"message": "[Quarkus.TermsWarnings] Consider using 'might (for possiblity)' or 'can (for ability)' rather than 'may' unless updating existing content that uses the term.", "location": {"path": "docs/src/main/asciidoc/logging.adoc", "range": {"start": {"line": 248, "column": 43}}}, "severity": "WARNING"}
|
Linting with Vale:
docs/src/main/asciidoc/logging.adoc#L248
[vale] reported by reviewdog 🐶
[Quarkus.TermsSuggestions] Depending on the context, consider using ', which (non restrictive clause preceded by a comma)' or 'that (restrictive clause without a comma)' rather than 'which'.
Raw Output:
{"message": "[Quarkus.TermsSuggestions] Depending on the context, consider using ', which (non restrictive clause preceded by a comma)' or 'that (restrictive clause without a comma)' rather than 'which'.", "location": {"path": "docs/src/main/asciidoc/logging.adoc", "range": {"start": {"line": 248, "column": 68}}}, "severity": "INFO"}
|
Linting with Vale:
docs/src/main/asciidoc/logging.adoc#L248
[vale] reported by reviewdog 🐶
[Quarkus.Spelling] Use correct American English spelling. Did you really mean 'formatters'?
Raw Output:
{"message": "[Quarkus.Spelling] Use correct American English spelling. Did you really mean 'formatters'?", "location": {"path": "docs/src/main/asciidoc/logging.adoc", "range": {"start": {"line": 248, "column": 168}}}, "severity": "WARNING"}
|
Linting with Vale:
docs/src/main/asciidoc/logging.adoc#L250
[vale] reported by reviewdog 🐶
[Quarkus.TermsWarnings] Consider using 'therefore' rather than 'Thus' unless updating existing content that uses the term.
Raw Output:
{"message": "[Quarkus.TermsWarnings] Consider using 'therefore' rather than 'Thus' unless updating existing content that uses the term.", "location": {"path": "docs/src/main/asciidoc/logging.adoc", "range": {"start": {"line": 250, "column": 1}}}, "severity": "WARNING"}
|