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
[RFC]: extend MoyaBot to automatically follow up with issues & PRs where we're waiting for author response #1071
Comments
I really like this - though looking at fastlane-bot's repository, I have no idea how it would work implementing it. Does anybody have an idea, or should we go ahead and file an issue there? |
Peril does look really nice, let's wait for that to be usable |
Just came across this: https://github.com/probot/stale 🤔 |
This one is really cool! I think we can try to do it. What do you think, @Moya/contributors? Edit: Also, can we do it using MoyaBot? |
Sure, this would be really nice! Re: MoyaBot: I think you should ping @ashfurrow to ask if that's possible. :) |
We just merged it in on CocoaPods - CocoaPods/CocoaPods#6745 - I can't really compete with GitHub on resources for this kind of project, so I wouldn't wait on peril |
Ye, and this one is looking cool. The question is if we want/can use MoyaBot for that. Also, we would need someone to do a PR with config. Let's just start with something and we will adjust values in a PR if needed. |
😆 |
I was talking with Ash and we've figured that MoyaBot would need a write access so it would be better to use different bot or someone else's access token. Thus, it might be better just to install this stale bot and go from that. I'm gonna do a PR with some default values for config so we can discuss it there 👍 |
Awesome! 👊 |
I've opened a PR#1101 with the config, please let me know what do you guys think! What we need to get this bot running:
Is there something I didn't think about? 🤔 |
That sounds good to me. Thanks for figuring this out!!
On to-do 2, let's wait to add labels until we've iterated on the bot, just
in case we end up not liking it
…On Thu, May 25, 2017 at 8:21 AM Łukasz Mróz ***@***.***> wrote:
I've opened a PR#1101 with the config, please let me know what do you guys
think!
What we need to get this bot running:
- #1101 <#1101> merged.
- Re-visit issues that might have need a label to not flag it.
- Install the bot & iterate with the feedback.
Is there something I didn't think about? 🤔
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1071 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ADo1dFodJSG0iR_dKP_AKOVN3RAbcAP6ks5r9Zx2gaJpZM4NQHMU>
.
|
I've revisited some issues anyways - I've flagged most of documentation ones to leave it as it is, because it is important that we have these open and add it in the future. If you guys want to make last review, I'm gonna try to launch the bot tomorrow morning or so. |
@sunshinejr thanks for taking the time! ❤️ |
Bot added - let see how it goes :D |
@sunshinejr The configuration looks correct to me. Do you think its a permission issue considering it can comment on issues but not close them? The link in the EDIT: I created #1157 in case |
👍 nice catch, @SD10. Can you take care of that issue and see if this PR helped? Because from the permission side it looks like it has everything it needs: |
@sunshinejr Thanks for checking the permissions. This change is already merged in. It may take some time to determine if this changes anything. There's little and no official documentation on when probot will scan the issues. Some info here if anyone is interested. I'll continue watching this until it's resolved. |
Great, thank you Steven! 🍻 |
I think the problem is with our label having spaces. |
@SD10 That's exactly what I was testing pedrovereza/stale-bot-test#2 😄 |
@pedrovereza You read my mind! I was going to ask you to please test this there 👍 |
@pedrovereza What do you think about changing our label to one without spaces? The support at stale looks limited. I don't think they'll be merging that PR anytime soon. |
@SD10 I'm ok waiting a bit longer for them to take a look at the PR. But if we decide to change the label, we need to revisit all the ones currently marked by the bot. @sunshinejr @AndrewSB @BasThomas thoughts? 😄 |
This should be as easy as changing the name of one label & updating the StaleBot config, correct? If so, and if its gonna fix the bot, I am up for the change 👍 I also don't mind waiting few days before we make a change, but seeing from the Issues/PRs it might be quite some time before they merge it. |
@sunshinejr We would probably have to resolve all the current issues manually. Updating a label is an action that will update the last modified date of an issue. Issues that are currently marked as It would work immediately for all issues not already marked. |
But if we rename the label to something without spaces, then the update date shouldn't change (since it is the same label object but a different name) and if we change name of the label in the config, the bot should catch that one as well, right? For instance, we could change label name to |
The API has an "updated_at" for issues. This might (?) be updated when a label changes? I'm not sure. |
We can try changing the label name and bot config, worst case we'll have to close them manually (which was the original plan anyway) 😄 |
Well, we can (probably) make sure it will work! 😄 I've just tested it on my test repo. My steps to reproduce:
(Note: I was using this request to test it.) So if the data is correct and I didn't miss anything, whenever we change the label, the That was fun! 😄 |
I actually think it does the query using the I think that PR should've taken effect immediately, so I'm thinking there's a stateful issue. |
It looks like probot has stopped slacking off 😅. |
The issue was created to integrate the bot, and because it is both integrated and working now, there is a reason to close it 😄 We can create another issue for things related to this bot if something pops up. Thanks everyone for feedback & help! 🎉 |
I saw that over at Fastlane, there's actually another step locking an issue some time after an issue has been closed. Would it be possible to add this as well? And then maybe add another comment before locking that asks to create a new issue? |
Is that also using probot/stale, @BasThomas? I think we had this conversation here at #1101 and it wasn't possible, but using another plugin could work (though I don't know if we can do it without deploying it). |
Ah. No, it uses their Fastlanebot. Let's hold off on this for now then; not that important |
I recently saw @KrauseFx's talk on scaling open source communities (great talk, I'd recommend it to everyone who's doing anything at all in open source). https://news.realm.io/news/tryswift-felix-krause-scaling-open-source-communities-github-management
One of my big take aways was fastlane's fastlane-bot. Which would automatically follow up with issues and close them if there was no response.
Right now, a couple of contributors have been manually doing that, but I think it would be awesome if we could add some of fastlane-bot's functionality to our bot so we don't have to politely close stale issues
Any thoughts from other contributors? Does anyone want to take a shot at implementing that for MoyaBot?
The text was updated successfully, but these errors were encountered: