Testing

Matthew Selsky Matthew.Selsky at twosigma.com
Wed Dec 6 19:22:42 UTC 2017


On Wed, Dec 06, 2017 at 12:25:14PM -0500, Eric S. Raymond via devel wrote:

> I have a different plan.  I always write doc patches as part of my
> change commits; my discipline is to prevent code and docs from getting out
> of sync in the first place.

Does everyone on the project do that?  This "policy" isn't listed in devel/hacking.txt.  If it's a project policy, then it should be.

We also don't have formal code reviews (before commit) since many devs push directly to "master".  So we can't enforce any policies to code before they get committed to master.

At some point, maybe soonish, can we stop pushing directly to master and instead push to branches (either in the main repo, or a personal fork) and then submit MRs and go through the review/approval workflow that's built into GitLab?


Thanks,
-Matt


More information about the devel mailing list