Skip to content

add Code of Conduct - #8

Merged
ctcpip merged 1 commit into
expressjs:masterfrom
bjohansebas:add-coc
Mar 17, 2025
Merged

add Code of Conduct#8
ctcpip merged 1 commit into
expressjs:masterfrom
bjohansebas:add-coc

Conversation

@bjohansebas

Copy link
Copy Markdown
Member

This adds the code of conduct so that all repositories that don't have this file yet will have it and it will be up to date. It also resolves https://github.com/bjohansebas/codemod/issues/12#issuecomment-2469038159.

@bjohansebas

Copy link
Copy Markdown
MemberAuthor

cc: @expressjs/express-tc

@jonchurchjonchurch left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I do prefer we use github's ability to inherit these kinds of files in cases where we'd just be duplicating or linking back to it anyways.

For packages which need a tweak, then yes we should include it in source. Otherwise Im happy to see the stock contributing and CoC just be inherited.

Comment threadCode-Of-Conduct.md

If an issue arises and you cannot resolve it directly with the parties
involved, you can report it to the Express project TC through the following
email: express-coc@lists.openjsf.org

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Are we sure that this email list is working/updated?

Comment threadCode-Of-Conduct.md
This is a foundation-wide team established to manage escalation when a reporter
believes that a report to a member project or the CPC has not been properly
handled. In order to escalate to the CoCP send an email to
coc-escalation@lists.openjsf.org.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Are we sure that this email list is working/updated?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

That email list is working, but I have not updated it. Right now it is a spam honey pot and I had to disable the moderation setting for myself on it because I couldn't check all of that.

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

I think you were referring to the comment above, because this one is from CPC.

@wesleytodd

wesleytodd commented Nov 15, 2024

Copy link
Copy Markdown
Member

Good call on the emails. Same comments I made here but I missed them in this PR.

Edit: started a slack convo with Ben to get these sorted out before we merge this.

@ctcpipctcpip left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please rename the file to: CODE_OF_CONDUCT.md

@bjohansebas

Copy link
Copy Markdown
MemberAuthor

Any update on the emails?

@wesleytodd

Copy link
Copy Markdown
Member

I dont have one now, but I can work on this tomorrow. It shouldn't take too much to get these updated.

@bjohansebas

Copy link
Copy Markdown
MemberAuthor

any update?

@wesleytodd

Copy link
Copy Markdown
Member

It is the right email, but we need something else in place for spam moderation because we should not all be in there checking it and it was getting nearly 1 email a day that was spam when I enabled the moderator setting for myself. This would be a good discussion to have on a TC meeting or working session to figure out a plan for.

@ctcpip

Copy link
Copy Markdown
Member

discussed at TC meeting today. we are merging since there are no actual changes to the CoC; there is nothing new, etc. we will create a separate issue to resolve the email

@ctcpip
ctcpip merged commit 8d859cc into expressjs:masterMar 17, 2025
@bjohansebas
bjohansebas deleted the add-coc branch March 18, 2025 13:10
Sign up for freeto 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.

5 participants

@bjohansebas@wesleytodd@ctcpip@UlisesGascon@jonchurch