-
Notifications
You must be signed in to change notification settings - Fork 15
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
[do not merge] Vale test 7 #248
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
||
This is the official Tezos implementation of Vale. | ||
|
||
It is in a chapter and an article. |
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 🐶
[Tezos.Terms] Do not use the term 'chapter'
|
||
This is the official Tezos implementation of Vale. | ||
|
||
It is in a chapter and an article. |
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 🐶
[Tezos.Terms] Do not use the term 'article'
|
||
It is in a chapter and an article. | ||
|
||
We have l1 and l2, which is smart rollups. |
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.
[Tezos.Substitutions] Use 'layer 1' instead of 'l1'
|
||
It is in a chapter and an article. | ||
|
||
We have l1 and l2, which is smart rollups. |
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.
[Tezos.Substitutions] Use 'layer 2' instead of 'l2'
|
||
It is in a chapter and an article. | ||
|
||
We have l1 and l2, which is smart rollups. |
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 🐶
[Tezos.Capitalize] Always capitalize 'Smart Rollups'
|
||
We have l1 and l2, which is smart rollups. | ||
|
||
We love smart rollups. |
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 🐶
[Tezos.Capitalize] Always capitalize 'Smart Rollups'
|
||
We love smart rollups. | ||
|
||
The term Smart Contracts should not be capitalized. |
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 🐶
[Tezos.DoNotCapitalize] Do not capitalize 'smart contract'
|
||
The term Smart Contracts should not be capitalized. | ||
|
||
What if I failed to capitalize smart rollups on a different line? |
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 🐶
[Tezos.Capitalize] Always capitalize 'Smart Rollups'
|
||
What if I failed to capitalize smart rollups on a different line? | ||
|
||
Adding another line about smart rollups. |
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 🐶
[Tezos.Capitalize] Always capitalize 'Smart Rollups'
|
||
Adding another line about smart rollups. | ||
|
||
Some other things about smart rollups. |
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 🐶
[Tezos.Capitalize] Always capitalize 'Smart Rollups'
No description provided.