Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Sep 17 14:42

    pbludov on auto-ecj

    (compare)

  • Sep 17 11:01

    pbludov on auto-ecj

    minor: formatting in wercker.sh infra: update sha1 for pjdbc to… Pull #10800: fix checkstyle vio… and 1 more (compare)

  • Sep 16 19:57
    pbludov commented #11
  • Sep 16 12:59
    dependabot[bot] edited #11
  • Sep 16 12:59
    dependabot[bot] edited #11
  • Sep 16 12:59
    dependabot[bot] synchronize #11
  • Sep 16 12:59

    dependabot[bot] on maven

    dependency: bump spotbugs-maven… (compare)

  • Sep 16 12:59
    dependabot[bot] edited #11
  • Sep 16 12:58
    dependabot[bot] edited #11
  • Sep 16 12:58

    pbludov on master

    config: remove old antrl refere… Issue #10781: temporary disable… doc: splits up the release note… and 7 more (compare)

  • Sep 08 11:11
    dependabot[bot] labeled #20
  • Sep 08 11:11
    dependabot[bot] opened #20
  • Sep 08 11:11

    dependabot[bot] on maven

    dependency: bump maven-javadoc-… (compare)

  • Sep 06 12:32

    pbludov on issue-10781-ban-spotbugs

    Issue #10781: temporary disable… (compare)

  • Sep 06 11:12

    pbludov on issue-10781-ban-spotbugs

    config: remove old antrl refere… Issue #10781: temporary disable… (compare)

  • Sep 06 07:52

    pbludov on issue-10781-ban-spotbugs

    (compare)

  • Sep 06 07:52

    pbludov on issue-10781-ban-sporbugs

    Issue #10781: temporary disable… (compare)

  • Sep 06 07:46

    dependabot[bot] on maven

    (compare)

  • Sep 06 07:46
    dependabot[bot] closed #17
  • Sep 06 07:46
    dependabot[bot] edited #17
developerhb
@developerhb
@romani @rnveach I have created PR checkstyle/checkstyle#8195 for issue #7728
developerhb
@developerhb
@romani @rnveach I am importing org.checkstyle.suppressionxpathfilter.AbstractXpathTestSupportin test classes for issue #7728. In Travis CI exec-maven-plugin fails due to this import. But, it passes in mvn clean verify on my local. What could be the reason? PR reference checkstyle/checkstyle#8195
developerhb
@developerhb
@romani I have imported another class and CI has passed.
developerhb
@developerhb
@romani I have created issue #8200.
Roman Ivanov
@romani
All students, please rebase your PRs over latest code, we did a release so all PRs that have new Checks or new Check properties - MUST rebase and upgrade version in code.
Roman Ivanov
@romani

Hi all, now Google shared results of who passed to GSOC 2020. You should have emails on result. I know that some of you will be very frustrated on not being selected. I will provide full explanation on why you are not selected. Please send me private message, do not hesitate to ask.
Akhil Singh
@akki111singh
Congrats to all who are selected!
Wilfried L. Bounsi
@wilcoln
Kudos to the GSoCers!
Roman Ivanov
@romani
You can send me gitter message or email
Kaustubh Dixit
@DXTkastb
Congratulations to all selected members for GSoC, happy coding! :)
Nick Mancuso
@nmancus1
Congratulations to all who were selected!
Gaurab Dasgupta
@gaurabdg
Thank you all mentors for giving us such a great learning experience and guidance!
iamhlbx
@HuGanghui
Thank you all mentors for giving me chance to participate in GSoC 2020 and work with Checkstyle!
Abhishek Kumar
@Abhishek-kumar09
Congratulations guys for getting into GSoC🙂
Shashwat Jaiswal
@shashwatj07
A heartfelt thanks to all mentors and congratulations to GSoCers!
Shrey Kumar Singh
@sulkykookie
Congratulations to all the GSoCers! 😁😁
Tan Wei Liang
@wltan
Congrats to those selected, all the best for your projects!
developerhb
@developerhb
Congrats to all those who are selected in GSoC!
Harsh Kukreja
@harsh-kukreja
.
developerhb
@developerhb
@rnveach I have commented in issue #7925. Please have a look.
developerhb
@developerhb
@romani Please have a look at issue #8200
Abhishek Kumar
@Abhishek-kumar09
How can I build and run checkstyle-contribution repo specially checkstyle-tester and verify their test cases.
rnveach
@rnveach
The read me for checkstyle-tester is in the read me and on the bottom of the page in github. You can find how to execute it at https://github.com/checkstyle/contribution/tree/master/checkstyle-tester#executing-diffgroovy
Abhishek Kumar
@Abhishek-kumar09
thanks @rnveach
developerhb
@developerhb
@rnveach @romani I am getting wrecker pipeline failed in checkstyle/checkstyle#8195
Roman Ivanov
@romani
it is green now
developerhb
@developerhb
Thank you @romani .
developerhb
@developerhb
@romani I am interested in issue #4944 and I have created PR checkstyle/checkstyle#8247 for the same. Currently the PR only includes Layout Implentations. It will be updated when the other functionalities are implemented. I also have some questions about other functionalities. Please have a look at them in the comments section of issue #4944.
developerhb
@developerhb
@romani I am working on issue #4944. I would like to know can we generate xpath query for DetailNode?
Roman Ivanov
@romani
I replied at issue
developerhb
@developerhb
@romani I have a question in comments section of issue #4944. Please have a look.
Roman Ivanov
@romani
you got response
developerhb
@developerhb
Thank you @romani
developerhb
@developerhb
@romani I have two new queries in comments section of issue #4944. Please have a look.
Kevin S. Clarke
@ksclarke
I'm trying to figure out how to control the indentation of System.err.println below. Checkstyle tells me it should be indented four more spaces to the right than the System.out.println. But, I want it indented like below (with the same number of indentations). Is there a setting for this? I'm not seeing anything on https://checkstyle.sourceforge.io/apidocs/com/puppycrawl/tools/checkstyle/checks/indentation/IndentationCheck.html that looks like what I want.
        s3Client.head(myBucket, myKey, head -> {
            System.out.println(head);
        }, error -> {
            System.err.println("error");
        });
The only custom config I have in my Indentation module currently is <property name="throwsIndent" value="8" />
Kevin S. Clarke
@ksclarke
I found an existing ticket in the issues queue for this (I believe), so I've just added my information there: checkstyle/checkstyle#7926 | The person who opened that ticket didn't follow through with the all the requirements for submitting a ticket.
Roman Ivanov
@romani
@ksclarke , Indentation Check is very problemtic, there are a lot of problems in it, please review our issue tracker and make sure your are on latest version of checkstyle
you are welcome to provide patch with fix to make it work
Roman Ivanov
@romani
@ksclarke , thanks a lot for detailed issue report, I made cleanup in issue and now it become approved
Kevin S. Clarke
@ksclarke
Okay, thank you, @romani
developerhb
@developerhb
@romani I have two queries in comments section of issue #4944. Please have a look.
Roman Ivanov
@romani
@developerhb , I replied to you
developerhb
@developerhb
@romani I have created PR checkstyle/checkstyle#8317. Please have a look.
Abhishek Kumar
@Abhishek-kumar09

@romani @rnveach

Should AnnotationArrayInit follow the same rulesets as arrayInit does. Or it should have different rulesets.

checkstyle/checkstyle#5951

developerhb
@developerhb
@romani I am getting CI failure in PR checkstyle/checkstyle#8317. Please help me in solving this.
Abhishek Kumar
@Abhishek-kumar09
How much you like the new look of github...
Roman Ivanov
@romani
I replied you in PR
UI is better from perspective of vertical space usage .... we just need to over come habit to use it in old way