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

Make auto PR comments more user-friendly #60

Closed
TomKristie opened this issue Aug 4, 2020 · 1 comment
Closed

Make auto PR comments more user-friendly #60

TomKristie opened this issue Aug 4, 2020 · 1 comment
Assignees
Labels
type: process A process-related concern. May include testing, release, or the like.

Comments

@TomKristie
Copy link
Contributor

TomKristie commented Aug 4, 2020

Dependent on PR commenting feature to be developed first (depends on #59)

Update the PR commenting mechanism to minimize the number of comments generated while minimizing the number of unchanged text in a comment.

This reduces the number of irrelevant lines to the user which increases readability.

In this example, the entire LICENSE file only changes the date, but the code comment block range is the entire file. The GUI displays all the changed and unchanged text given the code comment block range.
image

Ideally we'd want to reduce the unimportant text, while also minimizing the number of comments.

@TomKristie TomKristie added the type: process A process-related concern. May include testing, release, or the like. label Aug 4, 2020
@TomKristie TomKristie changed the title Make PR commenting more user-friendly Make auto PR comments more user-friendly Aug 4, 2020
@chingor13
Copy link
Contributor

Going to close this until we run into a use-case where this is enhancement is needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: process A process-related concern. May include testing, release, or the like.
Projects
None yet
Development

No branches or pull requests

2 participants