-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Documentation] Add more rules to the contribution guide + Fix CMS cookbook #9189
Conversation
@@ -132,6 +132,10 @@ Language Standards | |||
The Vitamins are in my Fresh California Raisins | |||
|
|||
* Do not use `Serial (Oxford) Commas`_; | |||
* Please use appropriate, informative, rather formal language; | |||
* Do not place any kind of advertisements in the documentation; | |||
* The documentation should be neutral, without judgements, opnions. Make sure you do not favor anyone, |
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.
Typo: opinions
@@ -132,6 +132,10 @@ Language Standards | |||
The Vitamins are in my Fresh California Raisins | |||
|
|||
* Do not use `Serial (Oxford) Commas`_; |
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.
How did that happen? :(
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.
Needs to be rebased to target 1.0 branch.
2dec545
to
75a0ddf
Compare
Thanks Magdalena Banasiak! 🥇 |
Thank you! 🍾 |
Added some statements regarding the neutrality of Sylius Documentation to the contribution guide.