search

nodejs / email

MX server management for iojs.org (and eventually nodejs.org)
MIT License
61 stars 77 forks source link

Merge & Deploy process #255

Open legendecas opened 1 month ago

legendecas commented 1 month ago

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?

UlisesGascon commented 1 month ago

Until we feel comfortable with the automation (proposal in https://github.com/nodejs/email/pull/222) I agree that code owner requirement is a great idea :+1: