Skip to content

[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

Merged
merged 3 commits into from
Jun 29, 2024

Conversation

oliverklee
Copy link
Collaborator

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.

@oliverklee oliverklee added documentation developer-specific Issues that only affect maintainers, contributors, and people submitting PRs labels Jun 23, 2024
@oliverklee oliverklee requested review from sabberworm and JakeQZ June 23, 2024 07:34
@oliverklee oliverklee self-assigned this Jun 23, 2024
@oliverklee oliverklee force-pushed the docs/contributing-1 branch 3 times, most recently from e8ef096 to 0c010ff Compare June 23, 2024 09:58
Copy link
Collaborator

@JakeQZ JakeQZ left a 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

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Superfluous blank line.

Copy link
Collaborator Author

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/

Copy link
Collaborator

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.

Copy link
Collaborator

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.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@oliverklee oliverklee requested a review from JakeQZ June 24, 2024 08:54
Copy link
Collaborator

@JakeQZ JakeQZ left a 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.
@oliverklee oliverklee force-pushed the docs/contributing-1 branch from 9005573 to 654d143 Compare June 29, 2024 07:38
@oliverklee oliverklee requested a review from JakeQZ June 29, 2024 07:39
Copy link
Collaborator

@JakeQZ JakeQZ left a 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.)

@JakeQZ JakeQZ merged commit a572164 into main Jun 29, 2024
18 checks passed
@JakeQZ JakeQZ deleted the docs/contributing-1 branch June 29, 2024 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developer-specific Issues that only affect maintainers, contributors, and people submitting PRs documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants