Skip to content

Conversation

@bjohansebas
Copy link
Member

I’m trying to expand the ways to contribute to the project.

cc: @expressjs/triagers @expressjs/express-tc

@bjohansebas bjohansebas requested a review from Copilot May 29, 2025 19:13
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR overhauls and expands the project’s contributing guide with clearer structure and more detailed workflows.

  • Renamed and extended the introduction, governance, issue reporting, and PR sections.
  • Added step-by-step guidance for reproducible examples, security issues, and triaging.
  • Appended a Developer’s Certificate of Origin section.
Comments suppressed due to low confidence (2)

CONTRIBUTING.md:7

  • Grammar: 'a open governance model' should be 'an open governance model'.
Express has a open governance model. For information about the governance of the Express project,

CONTRIBUTING.md:105

  • The project name 'paquete' appears incorrect here; it should refer to Express.
a future release of paquete.

Copy link

@CBID2 CBID2 left a comment

Choose a reason for hiding this comment

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

You're off to a good start @bjohansebas. There some parts that need more details.

Signed-off-by: Sebastian Beltran <[email protected]>
…larify documentation issue reporting

Signed-off-by: Sebastian Beltran <[email protected]>
Signed-off-by: Sebastian Beltran <[email protected]>
@bjohansebas
Copy link
Member Author

@CBID2 Thanks, I’ve made the changes you requested, and I also made some others to further expand on the ideas for how to contribute.

Copy link

@CBID2 CBID2 left a comment

Choose a reason for hiding this comment

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

Looks good to me

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.

3 participants