Skip to content

Conversation

veryfancy
Copy link

@veryfancy veryfancy commented Oct 3, 2025

Description:

Introduce a parameter exempt-issue-types that ensures that issues from a list of types will not be marked stale or closed.

This is a blocklist counterpart to the allowlist option only-issue-types added in:

Related issue:

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

@veryfancy veryfancy requested a review from a team as a code owner October 3, 2025 21:52
@Copilot Copilot AI review requested due to automatic review settings October 3, 2025 21:52
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@maxwxyz
Copy link

maxwxyz commented Oct 4, 2025

@luketomlinson this is similar to #1255

Copy link
Contributor

@Bibo-Joshi Bibo-Joshi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi. As a heads-up, I see that main.ts was not updated, which will probably lead to the input being ignored. See #1298, were I discovered that for only-issue-types :)

@veryfancy veryfancy requested a review from Bibo-Joshi October 15, 2025 22:12
@veryfancy
Copy link
Author

@Bibo-Joshi Thank you for the heads-up. I updated main.ts to include input reading for exemptIssueTypes. I included onlyIssueTypes as well, in case this merges first (and because I suspect it might prevent a merge conflict if your PR merges first).

Copy link
Contributor

@Bibo-Joshi Bibo-Joshi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants