Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 08:28
    dependabot[bot] labeled #6
  • 08:28
    dependabot[bot] opened #6
  • 08:28

    dependabot[bot] on maven

    dependency: bump maven-dependen… (compare)

  • 08:27
    dependabot[bot] labeled #5
  • 08:27

    dependabot[bot] on maven

    dependency: bump commons-io fro… (compare)

  • 08:27
    dependabot[bot] opened #5
  • Jun 12 12:48

    pbludov on java16

    Attempt to make it Java16 compa… (compare)

  • Jun 12 10:00
    dependabot[bot] labeled #4
  • Jun 12 10:00
    dependabot[bot] opened #4
  • Jun 12 10:00

    dependabot[bot] on maven

    dependency: bump junit5-system-… (compare)

  • Jun 12 10:00
    dependabot[bot] labeled #3
  • Jun 12 10:00
    dependabot[bot] opened #3
  • Jun 12 10:00

    dependabot[bot] on maven

    dependency: bump maven-project-… (compare)

  • Jun 12 08:27

    pbludov on maven

    dependency: bump maven-project-… (compare)

  • Jun 03 07:14

    pbludov on issue-9146-java11

    infra: move few items from trav… dependency: bump junit-pioneer … dependency: bump maven.jacoco.p… and 30 more (compare)

  • 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)

Ullrich Hafner
@uhafner
I see. Note that the library to create the diff is independent of Jenkins and might be used in a plain maven plugin as well. Otherwise you would reinvent the wheel in CheckStyle (and in every other static analysis tool). And creating the diff is a non trivial task.
Dustin Singleton
@Singleton06
The comparison is not really the part that I'm too interested in. I basically want a way to take a snapshot of the current state of the project and make sure that any additional commits do not worsen the state of the project.
Therefore, if I have a way to output a suppression file that would cause a build of that same project to find no violations, that achieves the goal I'm looking for.
Ullrich Hafner
@uhafner
I understand, this is the simple way to implement such a feature (this is a approach that I started with as well). It is not refactoring-aware but will work in most other cases.
Roman Ivanov
@romani
code freeze for release 8.15 - no merges to master brnach
Roman Ivanov
@romani
Release is done
Jonathan Leitschuh
@JLLeitschuh
@romani Please check the private message I just sent you. I have an urgent issue I need to discusss with you and the checkstyle team privately.
Roman Ivanov
@romani
=======
code freeze .... while I am doing 8.16 release
====
Roman Ivanov
@romani
code freeze for 8.17 release
Roman Ivanov
@romani
Sheng Chen
@jdneo
Hi team, I have a question about the Eclipse Checkstyle plugin. The plugin use ASTVisitor to modify the AST, thus correct the Checkstyle violations. Is it possible to extract these visitors into a separate jar files for distribution? I'm trying to implement a Checkstyle extension in VS Code and I wish I can use these components as part of the dependencies instead of copying the code.
Sheng Chen
@jdneo
demo.gif
Above is the gif of the POC I have implemented for now
rnveach
@rnveach
@jdneo ASTVisitor is not a creation of checkstyle or eclipse-cs. It is made by eclipse and in their jars. You would have to find those and extract them to use them.
Sheng Chen
@jdneo
rnveach
@rnveach
@jdneo Just look at imports and class hierarchy. AbstractASTResolution is eclipse-cs, but it extends WorkbenchMarkerResolution which is eclipse's
Sheng Chen
@jdneo
Okay, maybe I haven't explained my problem clearly. I found that the ASTVisitor return by the method handleGetCorrectingASTVisitor() is very useful. Currently, if I want to implement the same logic, I need to copy the code from handleGetCorrectingASTVisitor(). It would be great if these components could be shared.
rnveach
@rnveach
I'm not seeing what your saying. If you want to use the code, you can include eclipse-cs' JARs and eclipse's JARs and instantiate ASTVisitor and use it as you wish.
vermasoft.com
@ashish2583
i am understanding that what i am doing because i am a new user so i do any thing whos i seen
rnveach
@rnveach
@ashish2583 If you are new to checkstyle and want to contribute, please look at.
http://checkstyle.sourceforge.net/beginning_development.html
http://checkstyle.sourceforge.net/contributing.html
Akshit Ostwal
@AkshitOstwal

Hi all I am Akshit Ostwal !!!!

I am new to open source community , i would love to start my journey from this community .

I am also willing to take part in GSoC 2019 as well .

I know JAVA ,C ,C++ , MySQL and right now learning python as well .

How can i contribute to this community ?? Please Guide me!!!!!

THANKS IN ADVANCE !!!!!

rnveach
@rnveach
@AkshitOstwal I recommend looking over http://checkstyle.sourceforge.net/beginning_development.html, setting up your IDE, and do a test build of the project and ensure you have no problems.
When you are ready, we can point you to an issue you can work on.
Roman Ivanov
@romani
======
code freeze till I finish 8.18 release
====
thanks a lot to all contributors!!
====
Roman Ivanov
@romani
code freeze ..... till end of release process for 8.19
Roman Ivanov
@romani
Release is technically done
But there are CI problems
cannot initialize module TreeWalker - Property 'cacheFile' does not exist, please check the documentation -
We need to resolve this before public announcement of release
Roman Ivanov
@romani
all CI issues are resolved.
release is done, public notification were sent.
Roman Ivanov
@romani
====
code freeze for 8.20 release
Roman Ivanov
@romani
release is done, thanks to all for contributions
Roman Ivanov
@romani
====
I requested werker to review issue with badge https://werckerpublic.slack.com/archives/C0B8P1XJ5/p1558531595017000
Roman Ivanov
@romani
request to Teamcity to resolve build issues - https://teamcity-support.jetbrains.com/hc/en-us/requests/2112906
Roman Ivanov
@romani
=======
code freeze till release is done
Roman Ivanov
@romani
release 8.22 is done, all CIs are green
===
Eugenio Romano
@eromano
Hi guys I have a problem in using checkstyle with a project wit multiple modules