Formalize and document PR review process and requirements for merge

meltano / handbook

Source for Meltano's public handbook (https://handbook.meltano.com/) and public issue tracker for process and policy proposals that will be documented there.

MIT License

3 stars 4 forks source link

Formalize and document PR review process and requirements for merge #301

Open cjohnhanson opened 2 years ago

cjohnhanson commented 2 years ago

This should include:

Detailed team-specific processes in the PR process docs for the engineering team
Contributor processes and requirements should be documented in the contributor guide
A basic PR template should be added to meltano/meltano with a checklist for merge requirements and links to the relevant documentation

WillDaSilva commented 2 years ago

https://github.com/meltano/meltano/pull/6407#issuecomment-1189416251

A note about keeping the PR description up-to-date should be added to the handbook & template.