Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
GRADLE-2888 Renamed new 'maxAllowedViolations' property to 'maxErrors' so as to match the already existing Checkstyle property, and therefore be more intuitive to Checkstyle users.

  1. … 8 more files in changeset.
Added maxAllowedViolations option to Checkstyle DSL. This option sets the maximum number of code violations that are tolerated before breaking the build or setting the invoked ANT task failure property.

  1. … 8 more files in changeset.
Polishing

+review REVIEW-6132

  1. … 1 more file in changeset.
GRADLE-3490: Delete temporary Checkstyle XML report

Signed-off-by: Martin Mosegaard Amdisen <martin.amdisen@praqma.com>

  1. … 1 more file in changeset.
GRADLE-3490: Checkstyle output when XML report is disabled

Generate the XML report in a temporary location when it is

explictly disabled. We still need to generate it, as it is

used to generate the HTML report.

Enable and update the corresponding test.

Signed-off-by: Martin Mosegaard Amdisen <martin.amdisen@praqma.com>

  1. … 1 more file in changeset.
Revert "GRADLE-3490: Build output when Checkstyle XML report is disabled"

This reverts commit 0aa085bf2e8781bbc38e84b4a9f0800d49a69a1b.

  1. … 1 more file in changeset.
GRADLE-3490: Build output when Checkstyle XML report is disabled

Move the XML report from build/reports to build/tmp when it is

explicitly not requested as build output.

Also update the test and enable it.

Signed-off-by: Martin Mosegaard Amdisen <martin.amdisen@praqma.com>

  1. … 1 more file in changeset.
Fix compatibility with Pmd<5

+review REVIEW-6081

Migrate `Pmd` task to Java

The task still uses an internal class written in Groovy to invoke an isolated Ant builder.

+review REVIEW-6081

    • -0
    • +110
    ./PmdInvoker.groovy
  1. … 3 more files in changeset.
Migrate `JDepend` from Groovy to Java

The task still uses an internal class written in Groovy to invoke an isolated Ant builder.

+review REVIEW-6081

    • -0
    • +49
    ./JDependInvoker.groovy
  1. … 3 more files in changeset.
Migrate `CodeNarc` task to Java

The task still uses an internal class written in Groovy to invoke an isolated Ant builder.

+review REVIEW-6081

    • -0
    • +76
    ./CodeNarcInvoker.groovy
  1. … 3 more files in changeset.
Convert `Checkstyle` task to Java

This commit converts the `Checkstyle` task to Java, but keeps the internal call mechanism, which is using `IsolatedAntBuilder`,

in Groovy, for practical reasons.

+review REVIEW-6081

    • -0
    • +88
    ./CheckstyleInvoker.groovy
  1. … 2 more files in changeset.
Removed some Java 6 specific behaviour that is no longer required.

    • -11
    • +0
    ./findbugs/FindBugsClasspathValidator.java
  1. … 5 more files in changeset.
Merge remote-tracking branch 'origin/release'

* origin/release:

Restored a constructor for an internal type, for backwards compatibility. The constructor is deprecated and will be removed in 3.0.

Restored some methods that previously leaked onto all decorated types. These are now deprecated and will be removed in 3.0.

Restored some behaviour in the `MavenDeployer.repository { ... } DSL that changed in a breaking way when the implementation of this interface was changed from Groovy to Java.

Adds four new build splits for speed.

Add release notes about API changes

Make DSL doc generation happy

Appease checkstyle

Restore binary compatibility with AntGroovydoc

Remove type bounds so we generate the same method signatures again

Add missing methods from Groovy -> Java conversion

Restore old Object only constructors

Remove setProperty from formerly-Groovy tasks

Improve integration with docs.gradle.org wrt CSS styles.

Review items for Manifest API fix

Remove unnecessary FileResolver injection in Ear

Restore lost access levels in codeQuality

Restore lost access levels in Jacoco plugin

Restore lost access level on EclipseClasspath.noExportConfiguration

Mention breaking change to RhinoWorkerHandleFactory

  1. … 25 more files in changeset.
Remove type bounds so we generate the same method signatures again

Add missing annotation to task property

+review REVIEW-5932

Annotate report task inputs and outputs

Added some more annotations. Also made sure to use interfaces instead of

concrete classes when referring to `ReportContainer` properties. This

allows better task property validation, as implementation details are

not mixed in.

+review REVIEW-5932

    • -0
    • +30
    ./FindBugsReportsInternal.java
  1. … 22 more files in changeset.
More strict Regex in FindBugsClasspathVaildator (#642)

- The old regex "findbugs-(.*)\\.jar" will match any jars start with findbugs-ANYSTRING.jar

Classpath like this ["findbugs-annotations-3.0.0.jar", "findbugs-3.0.0.jar",

"findbugs-jFormatString-3.0.0.jar"] will match first in the list then throw GradleException

Make the regex more strict to match only findbugs-3.0.0.jar

    • -1
    • +1
    ./findbugs/FindBugsClasspathValidator.java
  1. … 1 more file in changeset.
Add missing `GroovyObjectSupport`

+review REVIEW-5980

  1. … 9 more files in changeset.
Review items for codeQuality and jacoco Java conversion

+review REVIEW-5933

  1. … 5 more files in changeset.
CompileStatic subprojects/code-quality

Plus some Groovy to Java conversions

+review REVIEW-5933

    • -130
    • +0
    ./AbstractCodeQualityPlugin.groovy
    • -0
    • +198
    ./AbstractCodeQualityPlugin.java
  1. … 10 more files in changeset.
Reduce fixed cost of applying the Jacoco plugin

+review REVIEW-5933

    • -42
    • +0
    ./findbugs/FindBugsWorkerManager.groovy
    • -0
    • +44
    ./findbugs/FindBugsWorkerManager.java
  1. … 7 more files in changeset.
Reduce fixed cost of applying code quality plugins

First step: use @CompileStatic in all involved Groovy classes

+review REVIEW-5933

    • -1
    • +3
    ./findbugs/FindBugsWorkerManager.groovy
  1. … 16 more files in changeset.
Revert "Reduce exceptions in configuring some core plugins"

This reverts commit 24f3e5ca1d583c841b605fb9a976592b64bb2ea7.

- the goal is to improve DSL execution so that this kind of workarounds

aren't necessary

  1. … 2 more files in changeset.
Reduce exceptions in configuring some core plugins

- explicitly referencing `this` in the closures shortcuts the

MissingMethodException/MissingPropertyException control flow

  1. … 2 more files in changeset.
Move the CustomizableHtmlReport over to reporting.

+review REVIEW-5862

  1. … 7 more files in changeset.
Add missing Report related types to the documentation

- Rename CustomizableHTMLReport -> CustomizableHtmlReport to match other class names

- Split all reporting related types into a separate section to keep the core list short

    • -0
    • +42
    ./CustomizableHtmlReportImpl.java
  1. … 11 more files in changeset.
Introduced a multi-request worker, which represents a worker process that handles zero or more requests.

    • -1
    • +1
    ./findbugs/FindBugsWorkerManager.groovy
  1. … 24 more files in changeset.
Moved worker process classes into their own package.

    • -2
    • +2
    ./findbugs/FindBugsWorkerManager.groovy
  1. … 92 more files in changeset.
Renamed an interface.

    • -2
    • +2
    ./findbugs/FindBugsWorkerManager.groovy
  1. … 10 more files in changeset.