I noticed this too. Somehow i used the wrong channel to ask about it. Anyway, i think this is breaking any considerable collaboration approach, because every single PR must be refreshed immediately before merging. This will put off external contributors.
On Tue, Jan 5, 2021 at 10:42 PM David Davis <[email protected]> wrote: > I noticed for the repos that got updated, PRs must now be up to date > before merging. I think we previously had this disabled but I am guessing > it got enabled since it's the default when required status checks are > enabled? > > David > > > On Fri, Dec 11, 2020 at 7:26 AM Tanya Tereshchenko <[email protected]> > wrote: > >> FYI, I went through the following repos and enabled required checks >> (lint, test(pulp), test(docs), and test(s3)) where they were missing, for >> the master and for the release branches if such rules existed: >> - pulpcore >> - pulp_file >> - pulp_rpm >> - pulp_container >> - pulp_ansible >> - pulp_python >> - pulp_deb >> - pulp-2to3-migration >> - pulp-certguard >> >> >> On Wed, Dec 9, 2020 at 5:35 PM David Davis <[email protected]> wrote: >> >>> When we stopped using Travis, I disabled the required status checks in >>> Github for pull requests. To re-enable these required status checks, visit >>> the branch protection settings page for your plugin's repo. Configure a >>> branch protection rule and there should be a setting to require status >>> checks. >>> >>> David >>> _______________________________________________ >>> Pulp-dev mailing list >>> [email protected] >>> https://www.redhat.com/mailman/listinfo/pulp-dev >>> >> _______________________________________________ > Pulp-dev mailing list > [email protected] > https://www.redhat.com/mailman/listinfo/pulp-dev >
_______________________________________________ Pulp-dev mailing list [email protected] https://www.redhat.com/mailman/listinfo/pulp-dev
