Skip to content
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

Idea: Resolve or Delete Previous Comments #7

Open
mitchspano opened this issue Aug 3, 2022 · 2 comments
Open

Idea: Resolve or Delete Previous Comments #7

mitchspano opened this issue Aug 3, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@mitchspano
Copy link
Owner

If running the scan on every commit to a Pull Request, you may create many duplicative comments.

It would be great if the system could accept a boolean input parameter to determine if previously generated auto-comments should be deleted of resolve.

I am still not certain if Delete or Resolve would make more sense.

@mitchspano mitchspano added the enhancement New feature or request label Aug 3, 2022
@initdotd
Copy link

My preference is to lean towards resolving and letting the new scan create new comments as found. This way, the history is there for reflecting back or training purposes.

@codefriar
Copy link
Contributor

@mitchspano - Would love to know if any progress has been made here on resolving old comments? happy to chat with you about how I might help make it a reality.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants