Fix #49 - set auto-merge PRs on the priority queue at the Auto-Tester #50
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As there is no API for the
auto-tester
, here's a proposal to login as user, send the AutoMerge command to theauto-tester
and thus prioritize the PR.However, this requires one to be logged in at GitHub, so this PR first logs in at Github and then via OAuth at the
auto-tester
. As for both GitHub and the auto-tester CSFR protection exists, the authenticity tokens need to scraped from the response. For convenience (and as Vibe.d doesn't handle client-side cookies), I used the excellent requests library.The biggest downside to this approach is that it requires the "password" of the dlang-bot to be set as Heroku environment variable, but I don't know of any other way to do so without.
The following needs to be done as well:
dlang-bot
at the auto-tester and approve the OAuth confirmation dialogdlang-bot
for the auto-tester (requires to make a dummy PR, so that it can be approved)auto-tester
, s.t. it doesn't conflict with the dlang-bot (for example a merge triggered by the auto-tester wouldn't include the "merged-on-behalf-of" comment)As this is experimental (and a bit hard to test without messing with real PRs), I set the entire method to "soft-fail".
CC @braddr