Skip to content
This repository was archived by the owner on Mar 27, 2018. It is now read-only.

Duplicate fixes#566

Merged
gerring merged 2 commits intomasterfrom
duplicate_fixes
Feb 22, 2017
Merged

Duplicate fixes#566
gerring merged 2 commits intomasterfrom
duplicate_fixes

Conversation

@gerring
Copy link
Copy Markdown
Contributor

@gerring gerring commented Feb 22, 2017

No description provided.

Collection<String> done = new HashSet<>();
for (final String item : includes) {

if (done.contains(item)) continue;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

MAJOR At most one statement is allowed per line, but 2 statements were found on this line. rule

@gerring
Copy link
Copy Markdown
Contributor Author

gerring commented Feb 22, 2017

SonarQube analysis reported 1 issue

  • MAJOR 1 major

Watch the comments in this conversation to review them.

@gerring gerring merged commit c4125a8 into master Feb 22, 2017
@gerring gerring deleted the duplicate_fixes branch February 22, 2017 15:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant