Skip to content
This repository has been archived by the owner on May 20, 2023. It is now read-only.

Clarify what is meant by 'inactive' #34

Closed
astrofrog opened this issue May 23, 2017 · 3 comments
Closed

Clarify what is meant by 'inactive' #34

astrofrog opened this issue May 23, 2017 · 3 comments

Comments

@astrofrog
Copy link

It would be good to be explicit about what constitutes activity. For example, does adding a label constitute activity? Or changing a milestone? In my case I would like to be able to configure this to say that I specifically want activity to be linked either to a pushed commit or a comment, but ignore changes in labels, milestones and other such events. Is there a way this could be done?

@rallytime
Copy link
Contributor

Hi @astrofrog - I don't think there is any way to specify this right now. The bot currently uses the updated search option as seen here, which includes changes in milestones and labels as you've discovered.

I ran into confusion about this myself, as I wasn't sure if just a comment was the last event the bot looked for or not on issues, for example.

@bkeepers has asked me to get some docs together about this to clarify the updated confusion. However, I don't know if filtering these events would be possible to add. I think it's a nice idea. What do you think @bkeepers?

@bkeepers
Copy link
Contributor

Hey @astrofrog, great question. Per @rallytime's explanation, the plugin delegates to GitHub for determining updated time.

However, I don't know if filtering these events would be possible to add. I think it's a nice idea. What do you think @bkeepers?

It would be possible by fetching the timeline for each issue and filtering out the events you don't care about, but with the current implementation will require a lot of API requests. Switching to GraphQL (which I'd like to do sooner or later), might make it better. I'd be happy to review a PR that makes these changes, but probably won't be working on it myself anytime soon.

I think the doc updates that @rallytime is working on will at least help clarify current behavior.

@bkeepers
Copy link
Contributor

This was clarified in #41. Let me know if that helps!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants