Skip to content

Workshop of contribution for node - #67139

Open
EstebannSa wants to merge 2 commits into
nodejs:mainfrom
EstebannSa:my-first-contribution
Open

EstebannSa wants to merge 2 commits into
nodejs:mainfrom
EstebannSa:my-first-contribution

Conversation

@EstebannSa

Copy link
Copy Markdown

No description provided.

Signed-off-by: Esteban <salasesteban0228@gmail.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. permission Issues and PRs related to the Permission Model. test Issues and PRs related to Node.js core tests and test infrastructure. labels Oct 30, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Welcome to Node.js, and thank you for your first contribution!

Before review, please take a moment to read:

Please make sure every commit is signed off. For a first pull request, GitHub Actions require collaborator approval and Jenkins CI must be started by a collaborator or triager, so an initial wait is normal.

@Ethan-Arrowood Ethan-Arrowood added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Oct 30, 2026
@Ethan-Arrowood Ethan-Arrowood self-assigned this Oct 30, 2026
@Ethan-Arrowood

Copy link
Copy Markdown
Contributor

Good work @EstebannSa ! A couple things before we can merge this PR. Let me know if you need my help with any step. I also recommend looking at other PRs from the event to see how others accomplished things (for example #67125 is very similar to yours).

  1. Change the PR title to something that describes the change. You may use your commit message for inspiration.
  2. Add a description to the PR describing the change in slightly more detail.
  3. Squash your commits into one. Use the second commit message.

If you need any help just let me know!

Thanks for coming to the Node.js Code and Learn at JSConfMX!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. needs-ci PRs that need a full CI run. permission Issues and PRs related to the Permission Model. test Issues and PRs related to Node.js core tests and test infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants