Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Nov 22 20:18

    pbludov on issue-8018-operator-wrap

    Issue #8018: AST based operator… (compare)

  • Nov 22 18:22

    pbludov on issue-8018-operator-wrap

    Issue #8018: AST based operator… (compare)

  • Nov 22 16:47

    pbludov on issue-8018-operator-wrap

    Issue #8873: RequireThis: Incor… Issue #7609: Update doc for Fin… Issue #6915: InnerAssignment: f… and 2 more (compare)

  • Nov 22 15:00

    pbludov on issue-8018-operator-wrap

    Issue #8018: AST based operator… (compare)

  • Nov 22 14:59

    pbludov on issue-8018-operator-wrap

    Issue #8018: AST based operator… (compare)

  • Nov 22 14:59

    pbludov on issue-8018-operator-wrap

    Issue #8018: AST based operator… (compare)

  • Nov 22 13:05

    pbludov on issue-8018-operator-wrap

    Issue #8018: AST based operator… (compare)

  • Nov 22 12:56

    pbludov on issue-8018-operator-wrap

    Issue #8018: AST based operator… (compare)

  • Nov 22 12:22

    pbludov on issue-6722-missing-asterisk

    Issue #6722: new check MissingL… (compare)

  • Nov 22 12:19

    pbludov on issue-6722-missing-asterisk

    Issue #6722: new check MissingL… (compare)

  • Nov 22 12:07

    pbludov on issue-8018-operator-wrap

    Issue #8018: AST based operator… (compare)

  • Nov 22 11:49

    pbludov on issue-8018-operator-wrap

    Issue #8018: AST based operator… (compare)

  • Nov 22 07:27

    pbludov on issue-6722-missing-asterisk

    Issue #8537: update metadata fi… Issue #5148: Add MissingJavadoc… Issue #6722: new check MissingL… (compare)

  • Nov 21 12:03

    pbludov on issue-6722-missing-asterisk

    config: codeql-analysis.yml is … Issue #8951: Remove Buddy CI co… minor: fix check name in javadoc and 28 more (compare)

  • Nov 02 16:39

    pbludov on infra-semaphore

    (compare)

  • Nov 02 08:09

    pbludov on infra-semaphore

    config: make semaphore ci comma… (compare)

  • Nov 01 20:29

    pbludov on infra-semaphore

    config: make semaphore ci comma… (compare)

  • Nov 01 19:24

    pbludov on infra-semaphore

    config: codeql-analysis.yml is … Issue #8951: Remove Buddy CI co… config: make semaphore ci comma… (compare)

  • Nov 01 15:46

    pbludov on infra-semaphore

    config: make semaphore ci comma… (compare)

  • Nov 01 15:29

    pbludov on infra-semaphore

    config: make semaphore ci comma… (compare)

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
rnveach
@rnveach
@romani @anhminhtran235 Sometimes I get the ant build exception or an error in the pom, but I don't believe the one listed is the same. The main error looks to be eclipse can't modify C:\Windows\system32\target and I have no idea why it would be trying to write to the windows directory.
To me it saying the Windows directory is really wrong and needs to be identified. That may be the heart of the issues.
The 2 package errors can be ignored.
Roman Ivanov
@romani
@anhminhtran235 , please try to reinstall Eclipse, or install it differently
anhminhtran235
@anhminhtran235
@romani I have reinstalled Eclipse. The problem still persists.
The first 2 exception only appears if I import the project after running mvn install
I'm using apache-maven-3.6.3, Java jdk 11.0.7
anhminhtran235
@anhminhtran235
I'm trying IntelliJ to see if it is an Eclipse problem
anhminhtran235
@anhminhtran235
image.png
@romani @rnveach I'm able to run the DescendantTokenCheckTest. I think it is working now?
anhminhtran235
@anhminhtran235
image.png
The pom.xml file still has lots of error
anhminhtran235
@anhminhtran235
The pom.xml file still has lots of error
Seems like this is a common problem in IntelliJ? https://youtrack.jetbrains.com/issue/IDEA-221441?_ga=2.243179202.1354574129.1600282369-1386209117.1600282369
Should I be concerned?
Weird. pom.xml is fine now. I think it's working :)