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

Checkstyle: Fix naming violations in attachments (T-U) #4222

Conversation

ssoloff
Copy link
Member

@ssoloff ssoloff commented Oct 25, 2018

Overview

Fixes violations of the Checkstyle MemberName rule in the TerritoryAttachment, TerritoryEffectAttachment, UnitSupportAttachment, and UserActionAttachment classes.

Functional Changes

None.

Manual Testing Performed

None.

@codecov-io
Copy link

Codecov Report

Merging #4222 into master will decrease coverage by 0.02%.
The diff coverage is 50%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #4222      +/-   ##
============================================
- Coverage     22.61%   22.58%   -0.03%     
+ Complexity     6114     6107       -7     
============================================
  Files           870      870              
  Lines         71370    71370              
  Branches      11395    11395              
============================================
- Hits          16137    16118      -19     
- Misses        53168    53188      +20     
+ Partials       2065     2064       -1
Impacted Files Coverage Δ Complexity Δ
...tegy/triplea/attachments/UserActionAttachment.java 3.61% <20%> (ø) 1 <1> (ø) ⬇️
...ategy/triplea/attachments/TerritoryAttachment.java 40.37% <41.66%> (ø) 59 <24> (ø) ⬇️
...triplea/attachments/TerritoryEffectAttachment.java 58.97% <47.61%> (ø) 16 <3> (ø) ⬇️
...egy/triplea/attachments/UnitSupportAttachment.java 70.81% <68.18%> (ø) 51 <23> (ø) ⬇️
...engine/framework/map/download/FileSizeWatcher.java 0% <0%> (-71.43%) 0% <0%> (-4%)
...gy/engine/framework/map/download/DownloadFile.java 36.95% <0%> (-13.05%) 9% <0%> (-1%)
.../games/strategy/engine/ClientFileSystemHelper.java 55% <0%> (-2.5%) 10% <0%> (-1%)
.../src/main/java/games/strategy/net/nio/Decoder.java 72.82% <0%> (-2.18%) 24% <0%> (-1%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 699eb55...9286f36. Read the comment docs.

@RoiEXLab
Copy link
Member

Just mentioning it here because I don't think it has been noted elsewhere since we broke compatibility:
We should change the type of the collection implementations to the interface type to be able to change the type in other places as well.
Also we should have a look at all places in the code where a TODO comment contains the world incompatible.

@RoiEXLab RoiEXLab merged commit 19ce7c9 into triplea-game:master Oct 25, 2018
@ssoloff
Copy link
Member Author

ssoloff commented Oct 25, 2018

@RoiEXLab I added the items you noted to the task list of #2827.

@ssoloff ssoloff deleted the checkstyle-fix-naming-violations-in-territory-attachments branch October 25, 2018 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants