[MCHECKSTYLE-166] Drop @requiresDependencyResolution test

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Won't Fix
• Affects Version/s: 2.8
• Environment: N/A
• Assignee: Mark Struberg
• Reporter: Ernst de Haan
• Created: 10/Nov/11 02:09
• Updated: 15/Feb/19 16:14
• Resolved: 28/Feb/14 02:08

Description

Currently, the CheckstyleViolationCheckMojo class declares:

@requiresDependencyResolution test

However, that should not be necessary. Checkstyle works on source files, not on bytecode.

If this declaration would be removed, then this Checkstyle plugin should still work perfectly fine (I would expect without any further code changes).

The advantage would be that in our Continuous Integration pipeline I can skip the compile stage and immediately trigger the checkstyle stage. That would save us multiple minutes on the feedback roundtrip.

Issue Links

• is related to: MCHECKSTYLE-353 Don't resolve any dependencies