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.

Sign up for GitHub

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

Jump to bottom

Format using 100 character lines #416

Closed
ctubbsii opened this issue Apr 5, 2018 · 1 comment
Closed

Format using 100 character lines #416

ctubbsii opened this issue Apr 5, 2018 · 1 comment
Assignees
@ctubbsii
Projects
1.9.0 2.0.0

Comments

@ctubbsii
Copy link
Member

ctubbsii commented Apr 5, 2018

Formatting to 100 characters improves readability, especially for contributors with difficulty reading small fonts on their screens.

See this mailing list thread.
@ctubbsii ctubbsii self-assigned this Apr 5, 2018
ctubbsii added a commit to ctubbsii/accumulo that referenced this issue Apr 6, 2018
@ctubbsii
POM updates for formatting (apache#416)
caf58bc 
* Update to formatter-maven-plugin 2.7.2
* Update sortpom version
* Simplify formatting plugin profiles which require jdk8
* Use 100 character line limit
* Update ciManagement and issueManagement sections for next release
* Include rule to ignore @link/@see tags in javadocs
* Ignore long 'new *Map<>' lines, because they tend to be long with
  generics
ctubbsii added a commit to ctubbsii/accumulo that referenced this issue Apr 6, 2018
@ctubbsii
Fix line wrap and other checkstyle errors (apache#416)
2fc3986
ctubbsii added a commit to ctubbsii/accumulo that referenced this issue Apr 6, 2018
@ctubbsii
Fix line wrap and other checkstyle errors (apache#416)
b9b348c
ctubbsii added a commit to ctubbsii/accumulo that referenced this issue Apr 6, 2018
@ctubbsii
POM updates for formatting (apache#416)
fc20be3 
* Update to formatter-maven-plugin 2.7.2
* Update sortpom version
* Simplify formatting plugin profiles which require jdk8
* Use 100 character line limit
* Update ciManagement and issueManagement sections for next release
* Include rule to ignore @link/@see tags in javadocs
* Ignore long 'new *Map<>' lines, because they tend to be long with
  generics
ctubbsii added a commit to ctubbsii/accumulo that referenced this issue Apr 6, 2018
@ctubbsii
Additional wrapping fixes for checkstyle (apache#416)
fbaee3e 
Additional line-wrapping fixes for checkstyle for the master branch
@ctubbsii
Copy link
Member Author

ctubbsii commented Apr 6, 2018

Fixed in fbaee3e for master/2.0
Fixed in b8c19f8 for 1.9.0

I did see a ShellServletIT failure, but it appears that this isn't a new failure and is unrelated.

@ctubbsii ctubbsii closed this as completed Apr 6, 2018
@ctubbsii ctubbsii added this to Done in 1.9.1 Jun 14, 2019
@ctubbsii ctubbsii added this to Done in 1.9.0 Jun 14, 2019
@ctubbsii ctubbsii added this to Done in 2.0.0 Jun 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ctubbsii ctubbsii

Labels
None yet
Projects
No open projects
1.9.0
  
Done
2.0.0
  
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ctubbsii