-
Notifications
You must be signed in to change notification settings - Fork 467
Convert multi-assignments to single assignments #5608
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
Conversation
A number of classes contain multiple variables assigned on the same line. Break up these assignments to separate lines. Closes apache#3689
The build is failing due to unformatted files. If you run |
core/src/test/java/org/apache/accumulo/core/iterators/user/RegExFilterTest.java
Outdated
Show resolved
Hide resolved
shell/src/main/java/org/apache/accumulo/shell/commands/InsertCommand.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Thanks for the PR @ibilley7. I see you aren't listed on our "people" page, acknowledging contributors. If you wish to be added as a contributor to https://accumulo.apache.org/people/ , please open a pull request to add yourself at https://github.com/apache/accumulo-website/edit/main/pages/people.md and leave a reference to If you intend to be a regular contributor to Accumulo projects, please consider subscribing to our developer mailing list (https://accumulo.apache.org/contact-us/) and introducing yourself. (this message has been mostly copied and pasted from this comment) |
example contribution PR apache/accumulo#5608
A number of classes contain multiple variables assigned on the same line. Break up these assignments to separate lines.
Closes #3689