search

googleapis / python-spanner-sqlalchemy

Apache License 2.0
38 stars 28 forks source link

build: allow merging pull requests that are not up to date #374

Open olavloite opened 7 months ago

olavloite commented 7 months ago

Allow pull requests that are not up to date with main to be merged, as we otherwise have re-run all tests for each PR before merging, which takes too long.

parthea commented 7 months ago

@olavloite Please could you file bugs for the failing presubmits?

olavloite commented 7 months ago

@olavloite Instead of relaxing strict status check altogether, should we consider removing the long running checks from requiredStatusCheckContexts? This way atleast the basic checks can be validated?

Note that this change does not mean that the required checks do not need to pass. Only that the branch does not need to be up to date with all changes in main. That makes it easier to run the tests for multiple (dependency) pull requests in parallel, and then merge them all. That means that there is still a small possibility that different dependencies break each other, but that will then show up in the build of the main branch (and in any release pull request).