contribution

contribution copied to clipboard

Issue #818: do shallow clone while retaining checkout abilitiy to specific SHA

17

Resolves #818 This PR reuses the earlier PR on shallow clone PR, while reverted retaining checkout abilitiy to specific SHA. I tested this PR on main repo https://github.com/checkstyle/checkstyle/pull/14662, by following...

relentless-pursuit

Regression report displays exceptions inconsistently

3

Discovered at https://github.com/checkstyle/checkstyle/pull/14791#issuecomment-2053817456 Regression report displays exceptions inconsistently. Sometimes there is a full stacktrace for an exception: https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/8039a13_2024203306/reports/diff/elasticsearch/index.html#A139 Sometimes there is only a line of the exception name: https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/8039a13_2024203306/reports/diff/elasticsearch/index.html#A1 Both...

Lmh-java

config: new check ConstructorsDeclarationGrouping

1

part of https://github.com/checkstyle/checkstyle/pull/14749 add new module ConstructorsDeclarationGrouping

Zopsss

add new check: JavadocLeadingAsteriskAlign

1

added `JavadocLeadingAsteriskAlign` check to `checkstyle-tester/checks-only-javadoc-error.xml`

Zopsss

groovy Runtime error while genrating the report

15

I was trying to manually generate a report this is my command ``` D:\contribution\checkstyle-tester> groovy diff.groovy --localGitRepo D:\checkstyle --baseBranch issue-3502 --patchBranch issue-3502-revertlabel --config my_check.xml --listOfProjects projects-to-test-on.properties ``` but I got...

mahfouz72

diff:groovy: can't clone checkstyle repo in some directories on Windows

4

Identified at https://github.com/checkstyle/contribution/issues/839#issuecomment-1970188522 , When running diff.groovy with the checkstyle on my local, I got the following error: ```` Cloning into 'repositories\checkstyle'... error: unable to create file src/it/resources/org/checkstyle/suppressionxpathfilter/unnecessarysemicolonafteroutertypedeclaration/SuppressionXpathRegressionUnnecessarySemicolonAfterOuterTypeDeclaration.java: Filename too...

rnveach

diff.groovy: improve clone time by depth=1

1

It is not big deal for CIs, it was a problem for me on hotspot, clone did not finish even for 10 minutes.... so I was not able to reuse...

romani

minor: add openjdk21 to projects file for diff report generation

4

nrmancuso

https://github.com/checkstyle/checkstyle/commit/35e09408d068ab40aa31fd33b9589318d69970b9 This is example how builder generated content that violated line length validation and caused build failure Expected: wrap content to multiple lines as we do with issue titles

romani

Issue #273: converts diff.groovy to use Checkstyle's CLI

6

Issue #273 Supplemental Checkstyle PR: https://github.com/checkstyle/checkstyle/pull/12576

rnveach