You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As we already store all open issues in memory, simply close all issues labeled TODO where no matching comment was found.
Elaboration
The problematic part would be if the action is run on multiple branches with different comments, in that case, we would have to look through the code in all branches that the action currently cannot access. Using the previous approach of pulling files via the API would however allow this.
Perhaps allowing users to opt into this functionality would be the best approach, as we also don't want to remove issues that aren't fixed just because the comment is removed.
The text was updated successfully, but these errors were encountered:
Suggestion
As we already store all open issues in memory, simply close all issues labeled
TODO
where no matching comment was found.Elaboration
The problematic part would be if the action is run on multiple branches with different comments, in that case, we would have to look through the code in all branches that the action currently cannot access. Using the previous approach of pulling files via the API would however allow this.
Perhaps allowing users to opt into this functionality would be the best approach, as we also don't want to remove issues that aren't fixed just because the comment is removed.
The text was updated successfully, but these errors were encountered: