Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • May 04 15:59

    pbludov on issue-9146-java11

    Issue #9146: specify java versi… (compare)

  • May 04 15:56

    pbludov on issue-9146-java11

    infra: move PR description to a… infra: disable verify-no-except… Issue #9930: Resolved issues wi… and 3 more (compare)

  • May 03 13:29

    pbludov on issue-9146-java11

    Issue #9475: add code example o… config: upgrade sevntu-checksty… Issue #9474: add code example o… and 30 more (compare)

  • Apr 27 10:05

    pbludov on issue-9907-jdk16

    Issue #9907: set up Java16 build (compare)

  • Apr 27 09:09

    pbludov on issue-9907-jdk16

    infra: move xwiki validation to… infra: move pmd build-tools rep… Issue #9796: update MagicNumber… and 20 more (compare)

  • Apr 20 20:02

    pbludov on issue-9907-jdk16

    Issue #9907: set up Java16 build (compare)

  • Apr 20 19:35

    pbludov on issue-9907-jdk16

    Issue #8010: CommentsIndentatio… minor: fix formatting Issue #8989: Checkstyle parser … and 33 more (compare)

  • Mar 13 14:30

    pbludov on master

    minor: cleanup whitelist words Issue #7877: Removed redundant … Issue #5711: JavadocTagContinua… and 36 more (compare)

  • Feb 23 21:43

    pbludov on issue-9146-java11

    Issue #9146: specify java versi… (compare)

  • Feb 23 21:34

    pbludov on maven

    (compare)

  • Feb 23 21:31

    pbludov on issue-9146-java11

    minor: cleanup whitelist words Issue #7877: Removed redundant … Issue #5711: JavadocTagContinua… and 9 more (compare)

  • Feb 23 21:24

    pbludov on issue-9310-java11-inputs

    (compare)

  • Feb 23 08:28

    pbludov on issue-9310-java11-inputs

    Issue #9310: test inputs should… (compare)

  • Feb 23 07:03

    pbludov on issue-9310-java11-inputs

    Issue #9310: test inputs should… (compare)

  • Feb 23 06:52

    pbludov on issue-9310-java11-inputs

    Issue #9152: Record implementin… infra: add checkout_from suppor… dependency: bump archunit-junit… and 3 more (compare)

  • Feb 21 19:48

    pbludov on issue-9310-java11-inputs

    minor: cleanup whitelist words Issue #7877: Removed redundant … Issue #5711: JavadocTagContinua… and 1 more (compare)

  • Feb 21 19:40

    pbludov on issue-9310-java11-inputs

    Issue #9310: test inputs should… (compare)

  • Feb 20 15:01

    pbludov on issue-9310-java11-inputs

    Issue #9310: test inputs should… (compare)

  • Feb 20 14:47

    pbludov on issue-9310-java11-inputs

    Issue #9310: test inputs should… (compare)

  • Feb 20 09:54

    pbludov on issue-9310-java11-inputs

    #Issue #9310: test inputs shoul… (compare)

Andrei Paikin
@strkkk
@Abhishek-kumar09 it looks like some weird error in codecov itself, it shows 100% coverage on your changes but complains about some generated files
Abhishek Kumar
@Abhishek-kumar09
Yes, I think so. I will force push the changes so that it runs again.
Rudraksh Jhalani
@beingrudraksh
Hello everyone, I'm new to Checkstyle.org and I'm looking forward to contribute to the project as this would be my first time experience working on such type of a project. Really excited to start with!
Roman Ivanov
@romani
@beingrudraksh , please read https://checkstyle.org/beginning_development.html, as make your IDE be able to debug let us know we can find easy issues to make your introduction smooth
neeti
@neetisaharan
" Hi , I'm new and I would like to contribute, can someone get me started? "
Abhishek Kumar
@Abhishek-kumar09
@neetisaharan You have to first Setup your IDE before starting develpment, You could choose any, but personally I find IntelliJ much easy to get started. You can just clone your own fork of the repo, and intelliJ will do all the maven setup and imports for you. You could follow the step by step development guide for more insights.
please read https://checkstyle.org/beginning_development.html
Abhishek Kumar
@Abhishek-kumar09

Checkout some easy to get started with issues: https://github.com/checkstyle/checkstyle/issues?q=is%3Aissue+is%3Aopen+label%3Aapproved+label%3Aeasy

Documentation related issues to know better about the project in less time(Recommended): https://github.com/checkstyle/checkstyle/labels/gsoc%20first%20issue

Just tell everyone that you are on it, some issues have already those comments, please ignore those if they are posted 30+ days ago. They are open and you could chose it work.

Happy Hacking!

Abhishek Kumar
@Abhishek-kumar09
public class InputIndentationNew { //indent:0 exp:0
    void test() throws IOException  { //indent:4 exp:4
        BufferedReader bf =  //indent:8 exp:8
                new BufferedReader(  //indent:16 exp:16
                new InputStreamReader(System.in) {  //indent:16 <----- line 5
                    int a = 0; //indent:20
                }); //indent:16 exp:16
    } //indent:4 exp:4
}  //indent:0 exp:0
Is line 5 correctly indented or it should follow a basic offset or line wrapping indentation being on the next line.
Here, basicOffset: 4, LineWrapping: 8
Roman Ivanov
@romani
it is line wrap case, should be indented +8
Rudraksh Jhalani
@beingrudraksh
I'm facing a problem while setting up in Eclipse. No such window 'Maven Setup Plugin Connectors' appears after importing project and as a result, I'm not able to get the target/generated-sources/antlr folder.
Roman Ivanov
@romani
@rnveach , can you advice something on this Eclipse problem ? @beingrudraksh , please share version of Eclipse
Roman Ivanov
@romani
https://checkstyle.org/releasenotes.html#Release_8.37 , thanks a lot to all contributors
anhminhtran235
@anhminhtran235
@beingrudraksh I faced some problem with Eclipse too. Changing to IntelliJ works for me
Akash Mondal
@AkMo3
Hello everyone, I looking forward to contribute to this project. I am facing a bit trouble to understand how the entire codebase works. It would be extremely help if anyone could explain that.
Roman Ivanov
@romani
@AkMo3 , please read https://checkstyle.org/beginning_development.html , as you become able to debug, read https://checkstyle.org/writingchecks.html . All Checks are very independent and completely covered by Junit, so just put a break point and discover. Ask more specific question if still a problem.
Akash Mondal
@AkMo3
Thanks @romani , Would definitely ask if further questions.
Saksham Gupta
@saksham9656
hi @romani since you have gave me an issue but being a beginner i could not understand that issue would you please help me in it?
Roman Ivanov
@romani
Saksham Gupta
@saksham9656
Yes sir i have read it thank you. I will now try my best to solve it
@romani sir i want to tell you that i have started learning java so can i also contribute in it side by side?
Roman Ivanov
@romani
good, welcome to project, checkstyle is good project to start to lear java, we have bunch of each issues, to let you be productive even on junior level
Saksham Gupta
@saksham9656
Thank you sir i will definately work hard . Thanks @romani for the motivation
Aryan Bindal
@aryaniiit002
In Gsoc First Issues we need to make check's javadoc and edit xdoc examples from the scratch, if there any help/reference that i can use it .
Or there is nothing but to understand the check and then work on the issue ourselves.
Roman Ivanov
@romani
it is just to understand a Check, understand CLI of checkstyle, how to configure Check and run validation, how to do commit and pass review
8 replies
Vyom Yadav
@Vyom-Yadav
@romani Sir I am competent in java, should I learn XML and XPath also to contribute, or can still contribute with the skillset I possess. If yes then please mention some issues also as all the issues I discovered were using XML.
Roman Ivanov
@romani
Java is enough, all other you will learn during contribution
Please help with checkstyle/checkstyle#7564
There was PR with try to fix, but it was failed to complete
If you agree, please make comment "I am on it" in issue to let others know
Akash Mondal
@AkMo3
I have sent a PR checkstyle/checkstyle#8982 . But in this I am not able to correctly write the contents on config.xml and Test.java. Can anyone help me with this. Also can someone review this PR.
Ehtesham Khursheed
@ek08
Hello everyone, I'm new to Checkstyle.org and I'm looking forward to contribute to the project .Please help me to get started;)
Ehtesham Khursheed
@ek08
sudo apt-get install nautilus-open-terminal doesn't work for newer version(20.04) of ubuntu. The output for this command is Unable to locate package nautilus-open-terminal. Instead sudo apt-get install nautilus-extension-gnome-terminal is what works.
Roman Ivanov
@romani
@ek08 , from what page did yuo take this ?
you can reuse https://github.com/checkstyle/checkstyle/wiki/To-student-of-GSoC-2020 page , it is good even for general contributors
let us know when you finish setup and ready for debugging
Ehtesham Khursheed
@ek08

@ek08 , from what page did yuo take this ?

@romani Sir i took this from Reddit (r/gnome).The link for the same is https://www.reddit.com/r/gnome/comments/9a5u4m/how_to_install_nautilusopenterminal_on_xubuntu/. Thanks for the help.Would let you know as soon as I am ready for debugging.

Roman Ivanov
@romani
I meant what our/checkstyle page has a outdated content, to let me fix it
Ehtesham Khursheed
@ek08

let us know when you finish setup and ready for debugging

@romani Sir,I have finished the setup in IntelliJ.

Roman Ivanov
@romani
@ek08 , please help with checkstyle/checkstyle#7609 , there are already some changes done but work was abandoned. Please do comment in issue "I am on it" if you start working.
if you will have question on what to update in our code, please look at bunch already merged/fixed similar changes and assoisated code changes - https://github.com/checkstyle/checkstyle/issues?q=is%3Aissue+label%3A%22gsoc+first+issue%22+is%3Aclosed
Ehtesham Khursheed
@ek08
Please review #8994
Ehtesham Khursheed
@ek08
Please review #8996
Vyom Yadav
@Vyom-Yadav
@romani Hey, I am new to checkstyle, so should I install the latest version of Ubuntu as Ubuntu 12.04 LTS reached its regular End of Life on April 28, 2017. No more package updates, including security updates, will be accepted to the 12.04 primary archive. Prepare Development Environment in Ubuntu 12.04 is there but not for newer versions.
Vyom Yadav
@Vyom-Yadav
Also, I m unable to install Ubuntu 12.04 due to an error 'global name 'sig' is not defined'. I tried searching the net but no help so far.
Abhishek Kumar
@Abhishek-kumar09

@Vyom-Yadav You can literally install and do development setup for checkstyle on all versions of ubuntu. You can also setup development environment on windows or mac. OS doesn't matter. Yes, if your hardware supports, install only latest operating systems.

Prepare Development Environment in Ubuntu 12.04 is there but not for newer versions.

Actually, it works on all versions of ubuntu after 12.04. I myself use 20.04 and that guide is helpful.

Which OS do you primarily use?
Vyom Yadav
@Vyom-Yadav
64 bit
Abhishek Kumar
@Abhishek-kumar09
I mean, linux or windows or mac
Vyom Yadav
@Vyom-Yadav
windows