New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use diff
for checking PR body
#1000
Conversation
Success! The temporary job (see here) shows that the diff approach will indeed work. |
This reverts commit ad13800.
Note, I have manually force-pushed and reset |
Codecov Report
@@ Coverage Diff @@
## master #1000 +/- ##
=======================================
Coverage 92.91% 92.91%
=======================================
Files 67 67
Lines 3780 3780
=======================================
Hits 3512 3512
Misses 268 268
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
Ooohhhh.... issue/pull number 1000 🤩 |
And what a way to do it... Who would've guessed it would be on fixing automated procedures ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy to keep merging until we get something that works! 😜
Fixes #995
This tries to use
diff
instead to check whether the first 8 lines of the latest PR body text is equal to the first 8 lines of.github/utils/single_dependency_pr_body.txt
.This test is the basis for whether the
dependabot_updates
branch should be reset tomaster
(when it has been utilized to update dependencies) ormaster
should just be merged intodependabot_updates
(when any other PR is merged intomaster
).I've added a single commit here that will be reverted, but it implements a temporary CI test to check the new method.
It retrieves the PR body for the latest PR that should initiate resetting
dependabot_updates
, and it also retrieves a PR body for another random PR that should initiate the merge.Then it uses the
diff
approach for both and finally uses it within anif
-sentence as is intended.