I agree that this is painful (and you should remove it). We might want
to consider something like bors in the future.
On Fri, Mar 15, 2019 at 11:00 AM Thomas Scholtes <firstname.lastname@example.org> wrote:
If there are no objections I’m also going to remove the requirement to have the PR branch
based on master. I think it is unreasonable to have outside contributors continuously
rebasing or us clicking the “Update branch” button. (The latter also results in messed up
Am Donnerstag, den 14.03.2019, 11:28 +0100 schrieb Thomas Scholtes:
> Hey everybody,
> Currently we can’t merge PRs from outside contributors because the Google Cloud Build is
> required but cannot not be run for forks because of security. I will mark the Google
> Build as not required anymore but mark the CircleCI build as required.
> We should only merge PRs that do not affect the tests, like doc fixes. (E.g.
> https://github.com/radicle-dev/radicle/pull/560). We should quickly migrate the CI build
> from Google to CircleCI to invite more outside collaboration.
> - Thomas
-----BEGIN PGP SIGNATURE-----
-----END PGP SIGNATURE-----