-
Notifications
You must be signed in to change notification settings - Fork 212
Add ability to ignore issues with an assignee #172
Conversation
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.
We'd love to have this at github/linguist!
@christianbundy I think you'll need to add tests in test/schema.test.js
(and maybe test/stale.test.js
).
Thanks for the quick review @pchaigno! I've just added some tests, please let me know if there's any other test coverage you think would be appropriate. Happy to hear Linguist is using Stale, I've always had great experiences on that repo. ❤️ |
Great to hear! |
Anything I can do to help this along? I'd really love to use this to keep some issues from becoming stale. |
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 for doing this @christianbundy - and thank you for your patience 🙏
Thanks a lot! Looking forward to using this. |
I'm noticing this a bit late, but shouldn't it be |
Feel free to change it if you'd like, I followed the pattern of |
@JasonEtco When will this take effect on GitHub? If it's not in effect yet, I'll open a pull request. |
Nice! Let's keep it that way then. |
This gives maintainers the ability to assign themselves (or someone else) to keep an issue being marked as stale. My hope is that this will let individual maintainers take responsibility for issues which would otherwise be marked as stale. Heavily influenced by #88. Thanks for your work on this project!
View rendered README.md