-
Notifications
You must be signed in to change notification settings - Fork 144
[TASK] Start the contributor guidelines with the CoC #616
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
Conversation
e8ef096
to
0c010ff
Compare
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.
Seems mostly fine. Added some comments.
address, without explicit permission | ||
* Other conduct which could reasonably be considered inappropriate in a | ||
professional setting | ||
|
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.
Superfluous blank 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.
I'd like to keep the CoC as close to the template as much as possible to make upgrades easier. If we think that think need improving there, I suggest we create a PR at their repo instead:
https://github.com/EthicalSource/contributor_covenant/
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.
Seems this comment should be in the general discussion, rather than a reply to specific review comment.
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.
There's still a superfluous blank 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.
Done.
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.
There's still one remaining superfluous blank line.
This is the first part of #489. I'd like to add sections bit by bit to make reviewing easier and faster, and to allow us to get things online quickly. This file will be in sync with its counterpart in the Emogrifier project as much as possible.
9005573
to
654d143
Compare
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.
Every journey begins with a single step. (Or starting the engine.)
This is the first part of #489.
I'd like to add sections bit by bit to make reviewing easier and faster, and to allow us to get things online quickly.
This file will be in sync with its counterpart in the Emogrifier project as much as possible.