by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jun 21 08:37

    pbludov on issue-8325-wercker-jdk11

    (compare)

  • Jun 19 08:52

    pbludov on issue-8325-wercker-jdk11

    minor: fix TC violations dependency: bump ant from 1.10.… Issue #7671: Update doc for NoF… and 61 more (compare)

  • Jun 02 08:39

    pbludov on issue-8283-fix-import-tests

    (compare)

  • May 31 14:54

    pbludov on issue-8283-fix-import-tests

    Issue #8283: Make test inputs c… (compare)

  • May 31 13:30

    pbludov on issue-8283-fix-import-tests

    Issue #8283: set java.version t… (compare)

  • May 31 13:21

    pbludov on issue-8283-fix-import-tests

    Issue #8283: Make import test i… Issue #8283: ignore methods dep… Issue #8283: set java.version t… (compare)

  • May 31 12:59

    pbludov on issue-8283-fix-import-tests

    minor: fix TC violations dependency: bump ant from 1.10.… Issue #7671: Update doc for NoF… and 37 more (compare)

  • May 30 13:36

    pbludov on issue-8275-spotbugs-tests

    (compare)

  • May 30 11:19

    pbludov on issue-8275-spotbugs-tests

    Issue #8262: ignore jacoco inte… Issue #8275: verify tests with … (compare)

  • May 30 10:39

    pbludov on issue-8262-spotbugs-jacoco

    (compare)

  • May 30 10:25

    pbludov on issue-8275-spotbugs-tests

    Issue #7892: Add validation for… minor: fix bad line break Issue #7690: ClassDataAbstracti… and 9 more (compare)

  • May 29 16:49

    pbludov on issue-4937-drop-cdata

    (compare)

  • May 28 08:00

    pbludov on issue-8262-spotbugs-jacoco

    Issue #8262: ignore jacoco inte… (compare)

  • May 28 07:58

    pbludov on issue-8262-spotbugs-jacoco

    Issue #7980: Resolve Pitest Iss… Issue #8228: Update CIs to vali… Issue #8228: Clean up after Tra… and 8 more (compare)

  • May 19 07:08

    pbludov on issue-4937-drop-cdata

    Issue #7921: Resolve Pitest Iss… Issue #3786: enable JOSM in Tra… Issue #4937: Drop CDATA support… (compare)

  • May 17 09:54

    pbludov on issue-4937-drop-cdata

    Issue #7728: Update AbstractChe… Issue #7873: Resolve Pitest Iss… Issue #7804: Resolve Pitest Iss… and 1 more (compare)

  • May 16 08:20

    pbludov on issue-4937-drop-cdata

    Issue #4937: Drop CDATA support… (compare)

  • May 16 08:19

    pbludov on issue-4937-drop-cdata

    minor: fix TC violations dependency: bump ant from 1.10.… Issue #7671: Update doc for NoF… and 3 more (compare)

  • May 11 05:15

    pbludov on master

    doc: update link to moved javad… Issue #6332: Java9 try with res… Issue #8036: Update test inputs… and 2 more (compare)

  • May 03 08:19

    pbludov on master

    Issue #7833: Create code of con… Issue #3201: Allow to configure… dependency: bump forbiddenapis … and 22 more (compare)

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
@shashwat70
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
developerhb
@developerhb
@romani when will checkstyle get the discussions feature on github?
Roman Ivanov
@romani
we never find a reason why we need this
discussion in issue tracker is the same
developerhb
@developerhb
@romani I have a question in PR checkstyle/checkstyle#8317. Please have a look.
developerhb
@developerhb
@romani The CI is now green in PR checkstyle/checkstyle#8317
developerhb
@developerhb
@romani I have applied changes requested by you in PR checkstyle/checkstyle#8317
Abhishek Kumar
@Abhishek-kumar09
Abhishek Kumar
@Abhishek-kumar09

Just wondering... Is checktyle is made to be used only inside IDEs, I was just exploring github and found a repo where checkstyle is used as github-actions to give code analysis within the PR as by the bot. It was: https://github.com/nikitasavinov/checkstyle-action/pull/6/files

I found it really good, so shared.

rnveach
@rnveach
@Abhishek-kumar09 checkstyle is used inside maven and CLI, so its not just interactive environments. It is really for anything.
Abhishek Kumar
@Abhishek-kumar09
great @rnveach . Thanks for clarifying things.