Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jan 20 04:26

    pbludov on issue-9146-java11-ci

    infra: update docker image for … Issue #9146: switch to jdk 11 i… (compare)

  • Jan 19 18:42

    pbludov on issue-9146-java11-ci

    Issue #6320: added test case fo… minor: Modified FullIdent to su… Issue #7504: New Check UnusedLo… and 5 more (compare)

  • Jan 17 06:41

    pbludov on issue-9146-java11-ci

    Issue #10924: Update UnicodeCh… minor: Add required permissions… minor: Bump year to 2022 and 1 more (compare)

  • Jan 17 02:28

    pbludov on issue-9146-java11-ci

    Issue #9146: switch to jdk 11 i… (compare)

  • Jan 17 02:04

    pbludov on issue-9146-java11-ci

    Issue #10924: Update UnicodeCh… Issue #9146: switch to jdk 11 i… (compare)

  • Jan 16 20:35

    pbludov on issue-9146-java11-ci

    infra: adds headers to all gram… minor: remove unnecessary heade… Issue #9146: switch to jdk 11 i… (compare)

  • Jan 16 20:27

    pbludov on issue-9146-java11-ci

    Issue #9146: switch to jdk 11 i… (compare)

  • Jan 16 20:06

    pbludov on issue-9146-java11-ci

    Issue #9146: switch to jdk 11 i… (compare)

  • Jan 16 19:58

    pbludov on issue-9146-java11-ci

    Issue #9146: switch to jdk 11 i… (compare)

  • Jan 16 18:28

    pbludov on issue-9146-java11-ci

    Issue #9146: switch to jdk 11 i… (compare)

  • Jan 16 17:04

    pbludov on issue-9146-java11-ci

    Issue #9146: switch to jdk 11 i… (compare)

  • Jan 16 17:03

    pbludov on issue-9146-java11-ci

    Issue #9146: switch to jdk 11 i… (compare)

  • Jan 16 16:54

    pbludov on issue-9146-java11-ci

    Issue #9146: switch to jdk 11 i… (compare)

  • Jan 16 16:49

    pbludov on issue-9146-java11

    (compare)

  • Jan 16 16:44

    pbludov on issue-11172-runtime-exit

    (compare)

  • Jan 14 11:26

    dependabot[bot] on maven

    dependency: bump slf4j-simple f… (compare)

  • Jan 14 11:26

    dependabot[bot] on maven

    (compare)

  • Jan 14 11:25
    dependabot[bot] closed #15
  • Jan 14 11:25
    dependabot[bot] commented #15
  • Jan 14 11:25
    dependabot[bot] labeled #36
developerhb
@developerhb
@rnveach okay I will do that. Thank you.
Gaurav Punjabi
@Gaurav-Punjabi
@romani Please reply to my queries here
checkstyle/checkstyle#8110
checkstyle/checkstyle#5890
Demezhan Marikov
@ImmortalRabbit
If my branch has 2 commits before first review, is there any way how I can reduce it to 1 commit without creating new branch with a new pull request?
rnveach
@rnveach
@ImmortalRabbit squash the commits into 1. PR guidelines provides an example command.
Demezhan Marikov
@ImmortalRabbit
@rnveach thanks
Demezhan Marikov
@ImmortalRabbit

@rnveach @romani What kind of example can be provided for this

<module name="LeftCurly">
  <property name="ignoreEnums" value="false"/>
</module>

Issue: checkstyle/checkstyle#7582
Link to documentation: https://checkstyle.org/config_blocks.html#LeftCurly

please review
developerhb
@developerhb
@romani @rnveach I have rebased the PR checkstyle/checkstyle#7953 and all checks have passed. Please review.
Demezhan Marikov
@ImmortalRabbit
@rnveach @romani What could cause this errors? Is someone else having same issue? All my pull request is taking to much time to complete travis-ci tests and failing for 403, 501 reasons. Branches are up to date. I have just checked.
Caused by: org.eclipse.aether.resolution.ArtifactResolutionException: Could not transfer artifact org.junit.platform:junit-platform-commons:jar:1.6.2 from/to central (https://repo.maven.apache.org/maven2): Authorization failed for https://repo.maven.apache.org/maven2/org/junit/platform/junit-platform-commons/1.6.2/junit-platform-commons-1.6.2.jar 403 Forbidden
Gaurav Punjabi
@Gaurav-Punjabi
As the mentors are busy in this period, should I wait for my PR to get merged or should I keep solving issues?
Kaustubh Dixit
@DXTkastb
solving issues is better I guess :)
Demezhan Marikov
@ImmortalRabbit

Screen Shot 2020-04-18 at 16.38.46.png

@Gaurav-Punjabi

Gaurav Punjabi
@Gaurav-Punjabi
Got it, thanks 👍🏻
Gaurab Dasgupta
@gaurabdg
@rnveach Can you please look into PR #8008?
Roman Ivanov
@romani
@ImmortalRabbit , please ask question in PR, looks like no reason of such example config, you can skip it for now
developerhb
@developerhb
@romani @rnveach can you please guide me for writing XpathQuery in the Test after updating the check to log DetailAST. I am working on issue #7728.
Roman Ivanov
@romani
@developerhb , please take a look at other PRs like this, there bunch of examples
Demezhan Marikov
@ImmortalRabbit
@romani @rnveach Could you check my questions for this issue? checkstyle/checkstyle#952
Tan Wei Liang
@wltan
just for everyone's info, CI is taking a while to pass due to a backlog of jobs on appveyor: https://ci.appveyor.com/project/Checkstyle/checkstyle/history
rnveach
@rnveach
@wltan I tried turning on killing duplicate builds for appveyor. Hopefully that will help keep the queue down.
Demezhan Marikov
@ImmortalRabbit
@rnveach @romani Could you look to my pull request? There is merge conflict. Could you suggest me how to fix it? checkstyle/checkstyle#8052
Praveen Kumar
@pkpale
@rnveach can you please review it checkstyle/checkstyle#7942
Shrey Kumar Singh
@sulkykookie

Is there any issue with the VisibilityModifer? I tried running it with allowPublicImmutableFields as true on the given example in the doc

public class ImmutableClass
{
  public final ImmutableSet<String> includes; // No warning
  public final ImmutableSet<String> excludes; // No warning
  public final java.lang.String notes; // No warning
  public final BigDecimal value; // No warning

  public ImmutableClass(Collection<String> includes, Collection<String> excludes,
               BigDecimal value, String notes)
  {
    this.includes = ImmutableSet.copyOf(includes);
    this.excludes = ImmutableSet.copyOf(excludes);
    this.value = value;
    this.notes = notes;
  }
}

and according to the doc it should throw no warning but it does...

$ java -jar ../checkstyle-8.30-all.jar -c config1.xml Test1.java
Starting audit...
[ERROR] C:\Users\Shrey\Desktop\Checkstyle jar\visibilityModifier\Test1.java:3:37: Variable 'includes' must be private and have accessor methods. [VisibilityModifier]
[ERROR] C:\Users\Shrey\Desktop\Checkstyle jar\visibilityModifier\Test1.java:4:37: Variable 'excludes' must be private and have accessor methods. [VisibilityModifier]
[ERROR] C:\Users\Shrey\Desktop\Checkstyle jar\visibilityModifier\Test1.java:5:33: Variable 'notes' must be private and have accessor methods. [VisibilityModifier]
[ERROR] C:\Users\Shrey\Desktop\Checkstyle jar\visibilityModifier\Test1.java:7:20: Variable 'someValue' must be private and have accessor methods. [VisibilityModifier]
Audit done.
Checkstyle ends with 4 errors.
Demezhan Marikov
@ImmortalRabbit

@sulkykookie I am not sure, what is wrong here but you can check in VisibilityModifierCheckTest.java where you can find this test:

    public void testAllowPublicFinalFieldsInImmutableClass() throws Exception {
        final DefaultConfiguration checkConfig =
                createModuleConfig(VisibilityModifierCheck.class);
        checkConfig.addAttribute("allowPublicImmutableFields", "true");
        final String[] expected = {
            "12:39: " + getCheckMessage(MSG_KEY, "includes"),
            "13:39: " + getCheckMessage(MSG_KEY, "excludes"),
            "16:23: " + getCheckMessage(MSG_KEY, "list"),
            "34:20: " + getCheckMessage(MSG_KEY, "value"),
            "36:24: " + getCheckMessage(MSG_KEY, "bValue"),
            "37:31: " + getCheckMessage(MSG_KEY, "longValue"),
            "41:19: " + getCheckMessage(MSG_KEY, "C_D_E"),
        };
        verify(checkConfig, getPath("InputVisibilityModifierImmutable.java"), expected);
    }

I think running test case will help you to understand why your example doesn't work. InputVisibilityModifierImmutable.java is a file with examples for this Check.

Shrey Kumar Singh
@sulkykookie
I was just asking if the currently available example in the doc is incorrect
Demezhan Marikov
@ImmortalRabbit
Firnas Aleem
@AMFIRNAS
Hi
can I get an approval to this ?
I am on PR creation step :)
developerhb
@developerhb

@romani @rnveach I am generating diff report.

I am running following command
groovy diff.groovy --localGitRepo ../../checkstyle --baseBranch master --patchBranch 7728-updateabtractchecks-customimportorder --config my_check.xml --listOfProjects projects-to-test-on.properties

I am getting following error.

Caught: java.io.IOException: Cannot run program cmd in directory (directory of checkstyle-tester): Create Process error=5, Access is denied

Please help.

rnveach
@rnveach
@developerhb I recommend to start a google group and provide a log of entire run, push your directory as it is to your own branch and give us the branch URL. I don't recognize that type of error.
developerhb
@developerhb
You mean I should push all the changes to my branch and give the url of that branch.
Roman Ivanov
@romani
yes
Access is denied
please try to delete your contribution repo and clone it one more time
if still a problem , please debug a script in IDE or by println to get to know what command isfailing
Gaurab Dasgupta
@gaurabdg
@romani @rnveach Please review and merge #8180 as the parent PR depends on this and I cant push changes till then.
Roman Ivanov
@romani
CI is failing ...
Gaurab Dasgupta
@gaurabdg
@romani My bad, please review now.
Shashwat Jaiswal
@shashwatj07
please help me know why this build is failing checkstyle/checkstyle#7956
Abhishek Kumar
@Abhishek-kumar09
How can I suppress checks for online ci, because in PR checkstyle/checkstyle#8038 I had to write suppression file in MainTest.java which is extremely large, so I am getting methodLength and FileLength issues. I had wrote suppressions in suppressions.xml file du to which antrun-phase-verify is passing but online checkstyle tests are failed, Let me know the possible approach to solve this problem. All the other PRs of Indentation is dependent on this PR for their regression tests.
Gaurab Dasgupta
@gaurabdg
@romani @rnveach Can you please approve #8181 as CI is failing because of it.
developerhb
@developerhb
@rnveach As per your recommendation I have created a google group and provided the complete log for error in diff.groovy. Here is the link https://groups.google.com/forum/#!topic/checkstyle-tester-error/m176xggjywo
developerhb
@developerhb
@romani As per your advice, I cloned the contribution repo one more time, but it didn't help.
@romani The command failing is mvn -e --batch-mode site -Dcheckstyle.config.location=my_check.xml -Dcheckstyle.excludes= -Dcheckstyle.version=8.32-SNAPSHOT -Dcheckstyle.failsOnError=false