Where communities thrive


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

    pbludov on issue-9149-stack-overflow2

    Issue #9149: removed input caus… (compare)

  • Jan 07 10:57

    pbludov on issue-9149-stack-overflow2

    minor: fix sonar violation 'Ref… doc: releasenotes 8.39 [maven-release-plugin] prepare … and 4 more (compare)

  • Jan 07 07:33

    pbludov on issue-9149-stack-overflow2

    (compare)

  • Jan 06 17:28

    pbludov on issue-9149-stack-overflow2

    Issue #9149: workaround for sta… (compare)

  • Jan 06 16:35

    pbludov on issue-9149-stack-overflow2

    Issue #9149: workaround for sta… (compare)

  • Jan 06 16:06

    pbludov on issue-9149-stack-overflow2

    minor: fix sonar violation 'Ref… doc: releasenotes 8.39 [maven-release-plugin] prepare … and 4 more (compare)

  • Dec 27 2020 14:23

    pbludov on master

    Issue #5951: Incorrect warning … minor: update wercker.sh to use… dependency: bump org.eclipse.jg… and 18 more (compare)

  • Dec 27 2020 12:12

    pbludov on issue-8864-cmd-set

    (compare)

  • Dec 27 2020 12:11

    pbludov on issue-9021-fix-josm

    (compare)

  • Dec 27 2020 12:11

    pbludov on issue-9021-fix-josm

    Issue #9021: fix josm validation (compare)

  • Dec 27 2020 12:11

    pbludov on issue-9021-fix-josm

    (compare)

  • Dec 27 2020 12:11

    pbludov on issue-8951-remove-buddy

    (compare)

  • Dec 27 2020 12:10

    pbludov on issue-6722-missing-asterisk

    (compare)

  • Dec 27 2020 12:08

    pbludov on issue-9124-stack-overflow-parenpad

    (compare)

  • Dec 27 2020 11:23

    pbludov on issue-8018-operator-wrap

    (compare)

  • Dec 27 2020 09:47

    pbludov on issue-9101-fix-stack-overflow

    (compare)

  • Dec 27 2020 09:43

    pbludov on issue-4219-shadowed-imports

    Issue #4219: UnusedImports does… (compare)

  • Dec 27 2020 08:59

    github-actions[bot] on issue-4219-shadowed-imports

    Issue #8651: Checkstyle fails o… Issue #4219: UnusedImports does… (compare)

  • Dec 27 2020 08:34

    pbludov on issue-8018-operator-wrap

    Issue #8018: AST based operator… (compare)

  • Dec 27 2020 08:00

    pbludov on issue-9101-fix-stack-overflow

    minor: resolve sonar violation … minor: fix link to redirected t… Pull #9129: suppress sonar viol… and 8 more (compare)

rishinpandit09
@rishinpandit09
@Abhishek-kumar09 How to find out the forst good issue
*first good issue
Roman Ivanov
@romani
@rishinpandit09 , use any from https://github.com/checkstyle/checkstyle/labels/gsoc%20first%20issue where PR(pull request) is not created
Abhishek Kumar
@Abhishek-kumar09
Merry Christmas to Checkstyle community.
Aryan Bindal
@aryaniiit002
Merry Christmas to you too!
Roman Ivanov
@romani
Happy holidays !!!
Vyom-Yadav
@Vyom-Yadav
Merry Christmas.
Aryan Bindal
@aryaniiit002
Screenshot from 2020-12-26 17-55-45.png

Cannot resolve plugin edu.illinois:nondex-maven-plugin:1.1.1
what should i do?

@romani Please check this

Abhishek Kumar
@Abhishek-kumar09
try to reclone the project, and select jdk 11. Once everything is running properly, make your changes and create a PR
3 replies
Aryan Bindal
@aryaniiit002
Screenshot from 2020-12-27 07-45-58.png
@romani i rechecked development guide and everything is right
Abhishek Kumar
@Abhishek-kumar09
@aryaniiit002 what are your changes? seems like it is giving end of line error. and checkstyle is building and running properly that's why you are seeing checkstyle:ant-phase verify errors now.
7 replies
Have you made any PR, or any commit with these changes
Abhishek Kumar
@Abhishek-kumar09
don't do anything extra.
  1. Just clone the project
  2. right click on pom.xml and click maven and then select generate sources and update folders
  3. Wait for every running process on intelliJ to be completed/closed.
  4. Then start your development or do mvn clean verify, etc.
4 replies
Aryan Bindal
@aryaniiit002
Issue.png

[ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 3.814 s <<< FAILURE! - in com.puppycrawl.tools.checkstyle.internal.XdocsJavaDocsTest
[ERROR] testAllCheckSectionJavaDocs Time elapsed: 3.813 s <<< ERROR!
java.lang.Error: Error was thrown while processing src/main/java/com/puppycrawl/tools/checkstyle/checks/imports/ImportOrderCheck.java
at com.puppycrawl.tools.checkstyle.internal.XdocsJavaDocsTest.examineCheckSection(XdocsJavaDocsTest.java:209)
at com.puppycrawl.tools.checkstyle.internal.XdocsJavaDocsTest.testAllCheckSectionJavaDocs(XdocsJavaDocsTest.java:182)
Caused by: org.opentest4j.AssertionFailedError:

any thoughts other than my xdoc and javadoc code changes are not alike? bcoz i have a way to solve it, if that's the ONLY issue here
And about the image- i tried to follow the instruction shown in terminal

Vyom-Yadav
@Vyom-Yadav
Yes that's the only probelm
1 reply
Roman Ivanov
@romani
https://checkstyle.org/releasenotes.html#Release_8.39 is released, thanks a lot to all contributors
and sorry that I did not have enough time this month to accept a lot of your PRs, I will try to catch up in january
happy new year !!!
Aryan Bindal
@aryaniiit002
Happy new year !! to you too
Aryan Bindal
@aryaniiit002
https://github.com/checkstyle/checkstyle/issues/8805#issuecomment-696892989
if may i, i wanted to know if any changes were made after that?
and what is "checkstyle.checkstyle" CI ?
Vyom-Yadav
@Vyom-Yadav
CI is continuous integration system, checkstyle uses Travis CI, it is used for building and testing. https://checkstyle.sourceforge.io/ci-management.html
Siddharth Singh
@stryder2000
I have worked out on the examples for some configurations for checkstyle/checkstyle#7695. Since i am new to open source, it would be great if someone can briefly guide me on how to proceed next?
Vyom-Yadav
@Vyom-Yadav
Update Xdoc and Javadoc file
Siddharth Singh
@stryder2000
@Vyom-Yadav in the Javadoc file, we have to update the class documentation only, Right?
Vyom-Yadav
@Vyom-Yadav
Changes in javadoc and xdoc should be similar and in javadoc change the mutiliine comment present at top, you can locate the module to be changed over there.
that comment shall be changed
Siddharth Singh
@stryder2000
Yes okay thanks!
Aryan Bindal
@aryaniiit002
i would like to know if @rnveach is active these days?
Aryan Bindal
@aryaniiit002
can i work on Pitest child issues?
Akash Mondal
@AkMo3
Hello everyone. I had a doubt regarding how to get javadoc tags. I wanted to get @summary tag . I had written the following code but it does not work. Can anyone help me out what is the error and how to correct it. I think my regex might be wrong due to which tagMatcher.find() doesn`t work. The code I had written is
/**
     * This regexp is used to extract the javadoc tags.
     */
    private static final Pattern JAVADOC_BLOCK_TAG_PATTERN = Pattern.compile("\\s@(\\w+)");

    private void containsSummaryTag(DetailNode ast) {
        final Matcher tagMatcher = JAVADOC_BLOCK_TAG_PATTERN.matcher(ast.getText());
        while (tagMatcher.find()) {
            final String tagName = tagMatcher.group(1);
            if (tagName.equals("summary")) {
                log(ast.getLineNumber(), "TAG NAME: ", tagName);
            }
        }
    }
3 replies
Devesh Tulshyan
@Devesh225
Hey, I'm a newbie to open source but I've a good understanding of Java, and I'm learning about new algorithms and data structures. How can I start contributing?
1 reply
AniketDGiri
@AniketDGiri
Dear Community Members,
I am Aniket Giri from India. Currently, I am pursuing Masters in Geoinformatics from CSRE, Indian Institute of Technology Bombay (IITB). I have my bachelor's in Computer Science.
I am fond of the open-source community as it provides me an opportunity to keep my skills updated, build connections, and improve communications skills. Thus, I would love to contribute. I would be very grateful to get any directions to pursue and participate.
1 reply
Ayushman Sachan
@bossever
Hi all, I had a doubt regarding the "update doc" issues on github. The checkstyle.org website already contains few examples showing what code is ok and what is not, do we need to copy the same ones to Javadoc? Or create new examples?
2 replies
Ayushman Sachan
@bossever

I'm getting this error while trying to run Checkstyle 8.40 jar using the java com.puppycrawl.tools.checkstyle.Main -c google_checks.xml MyClass.java command.

Error: Unable to initialize main class com.puppycrawl.tools.checkstyle.Main
Caused by: java.lang.NoClassDefFoundError: picocli/CommandLine$ParameterException

Should I be using 8.39 release instead?

6 replies
Aryan Bindal
@aryaniiit002
can somebody pls upload PIT Runner 'Run/Debug Configurations' screenshot with details filled for 'importOrder' tests?
4 replies
Ayushman Sachan
@bossever
Hey everyone,
should I create custom config.xml files to run a single check on a file?
2 replies
Siddharth Singh
@stryder2000
I am not able to remove trailing whitespaces from my commits, it just seems that my commits end with a '\n' by default and thus i am getting this error message: "Issue #7695 : Update doc for RegexpMultiline\n" is invalid. Can someone help me with that please?
3 replies
rnveach
@rnveach
@aryaniiit002 I am not as active as I use to be. You can always reach out to any other admin who is active.
1 reply
Abhishek Kumar
@Abhishek-kumar09
@stryder2000 do:
git commit --amend
You will have a textEditor sort of thing, just remove any extra whitespace or newline from that commit and save and exit
then push the changes to github
Siddharth Singh
@stryder2000
@Abhishek-kumar09 Already tried that.. have tried multiple things since morning.. including pre-commit but this is not resolving.. read an article that mentioned this problem of automatic addition of \n in commit message with vs code. not sure if i should switch to some other IDE
Abhishek Kumar
@Abhishek-kumar09
@stryder2000 If you are using vscode text-editor then there will be problems in the future development. Checkstyle needs an Java IDE for development. It will be way better if you would use IDE. IntelliJ Idea is my personal preference, you could use any.
1 reply
rnveach
@rnveach
@Abhishek-kumar09 @stryder2000 I haven't seen the PR, but are you sure it is the newline causing issues? Checkstyle should work with multiple new lines on the end of the commit message. https://github.com/checkstyle/checkstyle/blob/master/src/test/java/com/puppycrawl/tools/checkstyle/internal/CommitValidationTest.java#L128-L129
the only thing to avoid is any text appearing after the newline.
Siddharth Singh
@stryder2000
'\n' was the only thing that looked like a problem earlier bcz it this thing was mentioned in the rules. Later @Abhishek-kumar09 suggested the space between the issue number and colon might be causing problems, and after that didn't saw the commit message format error
Abhishek Kumar
@Abhishek-kumar09
@rnveach, Actually I didn't see any character after newline at-all, but yes there was multiple lines in the commit message, but every character itself was on the single line.
After that, he renamed his commit message and everything gone superbly well.
Akash Mondal
@AkMo3
Can anyone help me with checkstyle/checkstyle#9174 ? I am facing some coverage issues.