Merge & Deploy process

Open
#255 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
25/100
Issue type
Feature
Clarity
Needs clarification
Activity status
Stale
Tech stack
github
Domain
ci-cd, release

Research direction

No files or tests are named. Start by reviewing the responsibility guidance linked in the issue and the repository's merge and deployment settings. Clarify whether the intended outcome is a code-owner merge requirement or automated deployment, then define an acceptance check for the selected process.

Written by the indexing model from the issue text.

Description

I think I have accidentally merged PRs in this repo that were expected to be merged by @nodejs/build-infra team (https://github.com/nodejs/email?tab=readme-ov-file#responsibility), due to the inciting big green "merge" button on the PR once been approved (sorry!!).

Can we can add code owner requirement on merging PRs, or automate the deploy process?

Dominant language
JavaScript
Stars
65
Forks
85
Avg merge
7d 16h
Merged PRs (30d)
1

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from nodejs/email

All issues in nodejs/email

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.