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)

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
When I run the command with checkstyle:check everything is OK and it scan the module I asked but when it runs during the phase mvn install it says a message: If you use inline configuration for rules, don't specify a configLocation
<plugin>
        <groupId>org.apache.maven.plugins</groupId>
        <artifactId>maven-checkstyle-plugin</artifactId>
        <version>${checkstyle-maven-plugin.version}</version>
        <executions>
          <execution>
            <id>validate</id>
            <phase>validate</phase>
            <goals>
              <goal>check</goal>
            </goals>
          </execution>
        </executions>
        <configuration>
          <sourceDirectories>
            <sourceDirectory>activiti-api-impl</sourceDirectory>
          </sourceDirectories>
          <configLocation>./checkstyle.xml</configLocation>
          <encoding>UTF-8</encoding>
          <consoleOutput>true</consoleOutput>
          <failsOnError>true</failsOnError>
          <linkXRef>false</linkXRef>
        </configuration>
      </plugin>

    </plugins>
  </build>

  <reporting>
    <plugins>
      <plugin>
        <groupId>org.apache.maven.plugins</groupId>
        <artifactId>maven-checkstyle-plugin</artifactId>
        <version>${checkstyle-maven-plugin.version}</version>
        <configuration>
          <configLocation>./checkstyle.xml</configLocation>
        </configuration>
      </plugin>
    </plugins>
  </reporting>
this is the conf I use and for what I got executions configured as I did should not even run checkstyle during install
Roman Ivanov
@romani
we need more details, and all versions, please create mail thread in our mail group
Eugenio Romano
@eromano
ok thanks I will do it
and please share all possible details
Eugenio Romano
@eromano
I will link the PR with the cahnge in the project so it's clear is an open source project
Roman Ivanov
@romani
and output of mvn -X ...
Eugenio Romano
@eromano
sure no problem
thanks
Eugenio Romano
@eromano
matrixbot
@matrixbot
BuZZ-dEE hi is possible to convert an old checkstyle configuration / xml file to a new version if for example there breaking changes from the old one to the new one?