-
Notifications
You must be signed in to change notification settings - Fork 35
Test vale on Pull Request. #69
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
base: main
Are you sure you want to change the base?
Conversation
@@ -3,6 +3,9 @@ | |||
|
|||
pseudoinstruction | |||
pseudo-instruction |
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.
🚫 [vale] reported by reviewdog 🐶
[Vale.Avoid] Avoid using 'pseudo-instruction'.
@@ -3,6 +3,9 @@ | |||
|
|||
pseudoinstruction | |||
pseudo-instruction | |||
CSr |
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.
🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'CSr'?
@@ -3,6 +3,9 @@ | |||
|
|||
pseudoinstruction | |||
pseudo-instruction | |||
CSr |
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.
[RedHat.Spelling] Verify the word 'CSr'. It is not in the American English spelling dictionary used by Vale.
@@ -3,6 +3,9 @@ | |||
|
|||
pseudoinstruction | |||
pseudo-instruction | |||
CSr |
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.
[riscv.SpellingRV] Verify the word 'CSr'. It is not in the American English spelling dictionary used by Vale.
@@ -3,6 +3,9 @@ | |||
|
|||
pseudoinstruction | |||
pseudo-instruction | |||
CSr |
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.
📝 [vale] reported by reviewdog 🐶
[RedHat.PascalCamelCase] Consider wrapping this Pascal or Camel case term ('CSr') in backticks.
@@ -3,6 +3,9 @@ | |||
|
|||
pseudoinstruction | |||
pseudo-instruction | |||
CSr | |||
vsr | |||
csr |
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.
[riscv.SpellingRV] Verify the word 'csr'. It is not in the American English spelling dictionary used by Vale.
@@ -3,6 +3,9 @@ | |||
|
|||
pseudoinstruction | |||
pseudo-instruction | |||
CSr | |||
vsr | |||
csr |
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.
[RedHat.Spelling] Verify the word 'csr'. It is not in the American English spelling dictionary used by Vale.
@@ -3,6 +3,9 @@ | |||
|
|||
pseudoinstruction | |||
pseudo-instruction | |||
CSr | |||
vsr | |||
csr |
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.
🚫 [vale] reported by reviewdog 🐶
[riscv.CapitalizeCSR] The acronym CSR must always be capitalized.
@@ -3,6 +3,9 @@ | |||
|
|||
pseudoinstruction | |||
pseudo-instruction | |||
CSr | |||
vsr | |||
csr | |||
|
|||
AsciiDoc is fully documented, and its documentation is actively maintained. This document contains some information on AsciiDoc markup to get you started. |
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.
📝 [vale] reported by reviewdog 🐶
[RedHat.SimpleWords] Use simple language. Consider using 'has' rather than 'contains'.
@@ -3,6 +3,9 @@ | |||
|
|||
pseudoinstruction | |||
pseudo-instruction | |||
CSr | |||
vsr | |||
csr | |||
|
|||
AsciiDoc is fully documented, and its documentation is actively maintained. This document contains some information on AsciiDoc markup to get you started. |
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.
📝 [vale] reported by reviewdog 🐶
[RedHat.TermsSuggestions] Depending on the context, consider using 'information about' rather than 'information on'.
Signed-off-by: Bill Traynor <[email protected]>
No description provided.