Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 15 06:55

    pbludov on issue-6916-parallel

    Issue #6209: activate no-error-… Issue #8790: All Regexp Checks … Issue #7637: Update doc for Equ… and 5 more (compare)

  • Oct 11 07:57

    pbludov on issue-6916-parallel

    Issue #6916: enable parallel ex… (compare)

  • Oct 11 07:54

    pbludov on issue-6916-parallel

    Issue #6916: enable parallel ex… (compare)

  • Oct 11 07:19

    pbludov on issue-6916-parallel

    Issue #6916: enable parallel ex… (compare)

  • Oct 11 07:15

    pbludov on issue-6916-parallel

    Issue #6916: enable parallel ex… (compare)

  • Oct 11 07:13

    pbludov on issue-6916-parallel

    Issue #6916: enable parallel ex… (compare)

  • Oct 11 05:16

    pbludov on issue-6916-parallel

    minor: remove redundant spotbug… Issue #6658: pitest increase mu… minor: apply TokenUtil#forEachC… and 15 more (compare)

  • Oct 02 10:05

    pbludov on master

    Issue #8864: use .ci/validation… dependency: bump spotbugs-maven… minor: remove redundant spotbug… (compare)

  • Oct 01 17:41

    pbludov on issue-8864-appveyor

    (compare)

  • Oct 01 07:03

    pbludov on master

    Issue #8839: update eclipse com… Issue #8842: Add rebase github … minor: fix control does not ret… and 20 more (compare)

  • Oct 01 06:52

    pbludov on issue-8864-cmd-set

    Issue #8864: test .ci/validatio… (compare)

  • Sep 30 14:53

    pbludov on issue-8864-appveyor

    Issue #8864: use .ci/validation… (compare)

  • Sep 30 14:38

    pbludov on issue-8864-appveyor

    dependency: bump junit-pioneer … Issue #8855: set up cirrus ci Issue #8855: fix master build o… and 3 more (compare)

  • Sep 30 08:22

    pbludov on issue-8855-cirrus

    (compare)

  • Sep 30 06:11

    pbludov on issue-8855-cirrus

    Issue #8855: fix master build o… Issue #8855: cirrus ci badge Issue #8855: re-enable wagon.ht… (compare)

  • Sep 30 05:06

    pbludov on issue-8855-cirrus

    Issue #8855: fix master build o… (compare)

  • Sep 30 04:57

    pbludov on issue-8855-cirrus

    Issue #8855: fix master build o… (compare)

  • Sep 30 04:47

    pbludov on issue-8855-cirrus

    Issue #8855: fix master build o… (compare)

  • Sep 30 04:46

    pbludov on issue-8855-cirrus

    Issue #8855: fix master build o… (compare)

  • Sep 29 20:12

    pbludov on issue-8855-cirrus

    Issue #8855: set up cirrus ci Issue #8855: MainTest is broken… (compare)

Sarthak Handa
@sarthakhanda

[ERROR] Failures:
[ERROR] CommitValidationTest.testCommitMessageHasProperStructure:148 Commit 628a550c80a9b1b449efd3cb3f53c6dbc9f40146 message: "Fixed bug\n" is invalid
Proper commit message should adhere to the following rules:
1) Must match one of the following patterns:
^Issue #\d+: .$
^Pull #\d+: .
$
^(minor|config|infra|doc|spelling|dependency): .*$
2) It contains only one line of text
3) Must not end with a period, space, or tab
4) Commit message should be less than or equal to 200 characters

The rule broken was: 1
[INFO]
[ERROR] Tests run: 3256, Failures: 1, Errors: 0, Skipped: 0
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 51.425 s
[INFO] Finished at: 2020-08-02T22:03:36+05:30
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.22.2:test (default-test) on project checkstyle: There are test failures.
[ERROR]
[ERROR] Please refer to /home/sarthakhanda/Desktop/checkstyle/target/surefire-reports for the individual test results.
[ERROR] Please refer to dump files (if any exist) [date].dump, [date]-jvmRun[N].dump and [date].dumpstream.
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException

i'm getting these errors while running mvn verify
Andrei Paikin
@strkkk
@sarthakhanda error message says what is wrong
Sarthak Handa
@sarthakhanda
I see that but i followed the exact pattern for the commit message
rnveach
@rnveach
@sarthakhanda you didn't as error message shows the commit message that is failing the rules
Rituparno Biswas
@roopbiswas
Hi everyone, I'm Rituparno! I really wish to start contributing to checkstyle. I've gone through the issues and found #8269 and #7562 quite intriguing. Please let me know if I can work on any of them.
Roman Ivanov
@romani
you just got a reply
just make a comment "I am no it" in issues, and you are welcome with PR
Rituparno Biswas
@roopbiswas
Cool! Thanks!
Roman Ivanov
@romani
@developerhb , please do not forget to claim bounty
developerhb
@developerhb
@romani can you please guide how can I claim bounty? What procedure should I follow?
developerhb
@developerhb
@romani I have submitted a bounty claim for issue #4944.
Roman Ivanov
@romani
accepted. Please do more fixes.
developerhb
@developerhb
Sure @romani I will do more fixes. Would you suggest me any issues to work on?
Roman Ivanov
@romani
@developerhb , please try checkstyle/checkstyle#6658
developerhb
@developerhb
okay @romani I will look into this issue.
Abhishek Kumar
@Abhishek-kumar09
please also review: checkstyle/checkstyle#8719
Shashwat Jaiswal
@shashwatj07
Hey everyone! How is it going?
@romani I was really busy since the past few days. Now I want to restart contributing. Can you suggest some issues?
developerhb
@developerhb
@romani I have questions in comments section of issue checkstyle/checkstyle#6658 . Please have a look.
Andrei Paikin
@strkkk
@shashwatj07 I have merged PR for checkstyle/checkstyle#8122 but there are other items, I think you can continue to work on them
Shashwat Jaiswal
@shashwatj07
okay thanks
Abhishek Kumar
@Abhishek-kumar09
Shashwat Jaiswal
@shashwatj07
Abhishek Kumar
@Abhishek-kumar09
developerhb
@developerhb
@romani need help with issue checkstyle/checkstyle#6658 . I have some questions there.
Roman Ivanov
@romani
@Abhishek-kumar09 , please send PR to add this tool to https://checkstyle.org/#Related_Tools
Abhishek Kumar
@Abhishek-kumar09

please send PR to add this tool to https://checkstyle.org/#Related_Tools

Sorry, which tool. Sorry I didn't get it.

Abhishek Kumar
@Abhishek-kumar09
Need a small help!
According to new rules, in input files we have to write a //ok or //violation at the end of every file. I just want to know what it (ok or violation) depicts? <br>
Sorry I could not found the pull request regarding this change to learn more about it.
Roman Ivanov
@romani
Sorry I didn't get it.
Strange .. I remember I saw message from you about gihub action repo for checkstyle , but I do not see it now

Sorry I could not found the pull request regarding this change to learn more about it.

checkstyle/checkstyle#8825 I do pre-step to ease update for you

Abhishek Kumar
@Abhishek-kumar09

I remember I saw message from you about gihub action repo for checkstyle , but I do not see it now

Oh! that one... Ok..., I will send a PR regarding it. :)
Here is that comment https://github.com/checkstyle/checkstyle/pull/8719#issuecomment-691179809

Abhishek Kumar
@Abhishek-kumar09
@strkkk I have tried to take main checkstyle repo as a reference for creating diff report in github-actions. Please have a look on commit Abhishek-kumar09/checkstyle@a990e21
Andrei Paikin
@strkkk
@Abhishek-kumar09 I responded. We had missed this item when we were implementing this feature this summer. Testing changes for this pipeline is a bit complicated
Abhishek Kumar
@Abhishek-kumar09

Strange .. I remember I saw message from you about gihub action repo for checkstyle , but I do not see it now

Here is that repo... https://github.com/nikitasavinov/checkstyle-action
I will make PR regarding adding it to related tools

anhminhtran235
@anhminhtran235

Hi guys, I'm trying to set up development environment in Eclipse using this guide: https://checkstyle.org/eclipse.html.

I have forked checkstyle, opened eclipse and Select File > Import > Maven > Existing Maven Projects > myForkedFolder.

It said 'importing' for a while, then finished but give me the below errors:

img.PNG

I'm using Windows. When I import, the "Setup maven plugin connectors" window didn't appear.

I tried "Suppress Eclipse Warnings", the errors are still there.

Could somebody help me?

Thank you.

developerhb
@developerhb
@romani travis and cricle ci are failing in PR #8829. Please help.
Roman Ivanov
@romani
@anhminhtran235 , try to reimport project again, and try to build it in terminal first mvn install before importing.
anhminhtran235
@anhminhtran235
@romani I just did. The problem persists
image.png
Here's the screenshot of the error in pom.xml
Do people usually have problems importing when they use Windows?
Roman Ivanov
@romani
usually not, you have some problem with cache file creation from maven ant plugin. Please find in log of your Eclipse workspace full exception stacktrace and share it, we need to see cause exception.
@rnveach , have you ever experience such problem in windows+eclipse ?
Abhishek Kumar
@Abhishek-kumar09
https://github.com/Abhishek-kumar09/checkstyle/pull/16/files
I experienced some unusual behaviour could you help.
Andrei Paikin
@strkkk
@Abhishek-kumar09 I responded
anhminhtran235
@anhminhtran235
@romani Here's the errors
img.PNG