<div dir="ltr"><div>A week or two ago on irc, @misa identified that unit tests which run on travis with crane prs had been failing for a long time. These unit tests are run on each PR that is submitted, but broken and ignored. We were interested in finding a better way to ensure that our required checks all must pass before a PR can be merged.</div><div><br></div><div>I want to get feedback on an idea to resolve this. Github has settings which restricts merging until the required checks are met. It allows you to:<br></div><div><br></div><div>- required review before merging</div><div>- reviewed status checks passing before merging. For us this would include unit test runs and docs builder runs</div><div><br></div><div>What if we enable ^ on the 'master' branch of all the dev maintained repos including:</div><div><br></div><div>devel, pulp-ci, pulp, pulp_file, pulp_puppet, pulp_rpm, pulp_docker, crane, ansible-pulp3, pulp_template, pulp_example, <a href="http://pulpproject.org">pulpproject.org</a>, pulp_python, pulp_deb, ansible-pulp3_systemd, pulp_ostree, ansible-pulp3_db</div><div><br></div><div>We have already enabled this on the 3.0-dev branch in the pulp repo maybe 6 months ago, so this would be more of the same.</div><div><br></div><div>What do you all think about this idea? What questions or alternatives are there? Should this be done via a pup?</div><div><br></div><div>Any feedback or ideas are welcome.<br></div><div><br></div><div>-Brian<br></div></div>