-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
Remove automerge stuff #610
Conversation
We will be using GitHub native automerge feature. We also need to remove some webhook events. The ones we still need are: pull_request closed and pull_request labeled For python/core-workflow#498 and #PyConUS #PyConUSChallenge
Codecov Report
@@ Coverage Diff @@
## main #610 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 10 6 -4
Lines 1010 356 -654
Branches 66 23 -43
==========================================
- Hits 1010 356 -654
|
tests/test_util.py
Outdated
to_iterate = self._getiter_return[url] | ||
for item in to_iterate: | ||
yield item | ||
return to_return |
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.
GH won't allow the suggestion, but we can combine these now:
-to_return = self._getitem_return[self.getitem_url]
-return to_return
+return self._getitem_return[self.getitem_url]
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.
Thanks, updated.
Combine return statements
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.
Thanks!
We will be using GitHub native automerge feature.
We also need to remove some webhook events.
The ones we still need are: pull_request closed and pull_request labeled
For python/core-workflow#498 and #PyConUS #PyConUSChallenge