Let .pulls accept just a username. #20
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Before this, it needed both a username and a repo. Check this out: ``` @threebean │ .pulls threebot │ threebean: (pulls <username[/repo]>) -- List the pending pull requests on github repos. @threebean │ .pulls what/is/this threebot │ threebean: Must be GitHub repo of the format <username/repo> or just <username> @threebean │ .pulls lmacken threebot │ threebean: One moment, please... Looking up lmacken. threebot │ threebean: @msabramo's "README.rst: syntax highlight + tweaks" lmacken/tbgrep#8 filed │ 4 days ago threebot │ threebean: @Qalthos's "Update action_hooks" lmacken/pyramid-openshift-quickstart#7 │ filed a year ago e @threebean │ .pulls fedora-infra threebot │ threebean: One moment, please... threebot │ threebean: @ralphbean's "Handle timezones." fedora-infra/bugzilla2fedmsg#4 filed 41 │ minutes ago threebot │ threebean: @ralphbean's "Remove unused imports." fedora-infra/fedmsg#300 filed 2 days │ ago threebot │ threebean: @pypingou's "Let the unorphan action call the unorphan API endpoint" │ fedora-infra/packagedb-cli#21 filed 3 days ago threebot │ threebean: @ralphbean's "Getting ready for the 1.2.0 release." fedora-infra/kitchen#5 │ filed 4 days ago threebot │ threebean: ... and 6 more. @threebean │ .pulls fedora-infra/fedmsg threebot │ threebean: @ralphbean's "Remove unused imports." fedora-infra/fedmsg#300 filed 2 days │ ago ```
|
I figure this will be more useful to |
| list(self.yield_github_pulls(username, repo)) for repo in repos | ||
| ], []) | ||
|
|
||
| # Reverse-sort by time (newest-first) |
There was a problem hiding this comment.
Maybe it's useful to also have a .stalepulls [/] which is sorted oldest-first, so you can just go through the list from old to new?
|
Looks good to me! 👍 |
ralphbean
added a commit
that referenced
this pull request
Nov 17, 2014
Let .pulls accept just a username.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Before this, it needed both a username and a repo. Check this out: