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

Multiple issues for a line should be combined #83

Open
bradleyfalzon opened this issue Apr 13, 2017 · 0 comments
Open

Multiple issues for a line should be combined #83

bradleyfalzon opened this issue Apr 13, 2017 · 0 comments

Comments

@bradleyfalzon
Copy link
Owner

In some cases one tool, or many tools report multiple issues for the same line. Currently, GopherCI posts multiple comments for the same line.

Should just be a matter of adjusting FilterComments or similar preprocessing step in or around

func (i *Installation) FilterIssues(ctx context.Context, owner, repo string, prNumber int, issues []db.Issue) (suppressed int, filtered []db.Issue, err error) {

screenshot 2017-04-14 07 25 45

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

No branches or pull requests

1 participant