Skip to content

(draft; discussion) Linting - #41

Closed
jcxldn wants to merge 2 commits into
masterfrom
feat/jcx/linting
Closed

(draft; discussion) Linting#41
jcxldn wants to merge 2 commits into
masterfrom
feat/jcx/linting

Conversation

@jcxldn

@jcxldnjcxldn commented Dec 6, 2020

Copy link
Copy Markdown
Member

Initial testing of using linting at build-time. Thoughts?

@codecov

codecovBot commented Dec 6, 2020

Copy link
Copy Markdown

Codecov Report

Merging #41 (616bffc) into master (efa74b2) will decrease coverage by 0.20%.
The diff coverage is 28.00%.

Impacted file tree graph

@@ Coverage Diff @@## master #41 +/- ##
============================================
- Coverage 14.58% 14.37% -0.21% 
Complexity 100 100 ============================================
Files 52 52 Lines 2167 2163 -4 Branches 339 340 +1 ============================================
- Hits 316 311 -5 - Misses 1836 1837 +1 
Partials 15 15 
Impacted FilesCoverage ΔComplexity Δ
...gdiner/stickyapi/bukkit/command/PluginCommand.java0.00% <ø> (ø)0.00 <0.00> (ø)
...mbdogdiner/stickyapi/bukkit/gui/ClickableSlot.java0.00% <ø> (ø)0.00 <0.00> (ø)
...ava/com/dumbdogdiner/stickyapi/bukkit/gui/GUI.java0.00% <ø> (ø)0.00 <0.00> (ø)
...iner/stickyapi/bukkit/particle/ParticleSystem.java0.00% <0.00%> (ø)0.00 <0.00> (ø)
...diner/stickyapi/bukkit/particle/shapes/Circle.java0.00% <ø> (ø)0.00 <0.00> (ø)
...dumbdogdiner/stickyapi/bukkit/util/ServerUtil.java75.00% <ø> (ø)4.00 <0.00> (ø)
.../dumbdogdiner/stickyapi/bukkit/util/SoundUtil.java0.00% <ø> (ø)0.00 <0.00> (ø)
...umbdogdiner/stickyapi/bukkit/util/StartupUtil.java0.00% <ø> (ø)0.00 <0.00> (ø)
...tickyapi/bungeecord/packet/PacketRegistration.java0.00% <ø> (ø)0.00 <0.00> (ø)
...diner/stickyapi/bungeecord/packet/SoundPacket.java0.00% <ø> (ø)0.00 <0.00> (ø)
... and 30 more

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 efa74b2...616bffc. Read the comment docs.

@ZachyFoxx

Copy link
Copy Markdown
Member

Closed because of #58

Sign up for freeto 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.

2 participants

@jcxldn@ZachyFoxx