Skip to content
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

(chore) Update COC to OpenJS #2496

Merged
merged 2 commits into from
Feb 23, 2024
Merged

(chore) Update COC to OpenJS #2496

merged 2 commits into from
Feb 23, 2024

Conversation

chrisgervang
Copy link
Collaborator

Based this on Electron's, which is another OpenJS project.

contributing/CODE_OF_CONDUCT.md Outdated Show resolved Hide resolved
* Gracefully accepting constructive criticism
* Focusing on what is best for the community
* Showing empathy towards other community members
We as members, contributors, and leaders pledge to make participation in our
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we need to add all this text when we are already referencing a master doc?

@ibgreen
Copy link
Collaborator

ibgreen commented Jan 16, 2024

@chrisgervang Can you --signoff your commit to make CI pass?

@ibgreen
Copy link
Collaborator

ibgreen commented Jan 27, 2024

@chrisgervang Can you signoff the commit (git ci --signoff) so that this passes CI?

@chrisgervang
Copy link
Collaborator Author

Actually, I'd like to take this in a different direction following some more discussion with OpenJS. They think we can just use a short CoC and link to theres.

Signed-off-by: Chris Gervang <chris@gervang.com>
Signed-off-by: Chris Gervang <chris@gervang.com>
@chrisgervang chrisgervang merged commit 55fb242 into master Feb 23, 2024
8 checks passed
@chrisgervang chrisgervang deleted the chr/coc branch February 23, 2024 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants