Where communities thrive


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

    pbludov on issue-6916-api

    (compare)

  • 05:09

    pbludov on issue-6916-checks-coding

    (compare)

  • 03:19

    pbludov on issue-6916-checks-naming

    Issue #6916: migrate tests to j… (compare)

  • 03:10

    pbludov on issue-6916-api

    Issue #6916: migrate tests to j… (compare)

  • 03:07

    pbludov on issue-6916-checks-coding

    Issue #6916: migrate tests to j… (compare)

  • 03:00

    pbludov on issue-6916-checks-coding

    Issue #6916: migrate tests to j… Issue #6916: migrate tests to j… Issue #6916: migrate tests to j… (compare)

  • 02:22

    pbludov on issue-6916-api

    Issue #7306: fix RequireThis fo… infra: move spellchecker to one… minor: remove obsolete jacoco s… and 12 more (compare)

  • 02:17

    pbludov on issue-6916-checks-modifier

    (compare)

  • 02:17

    pbludov on issue-6916-checks-blocks

    (compare)

  • Dec 07 18:08

    pbludov on issue-6916-checks-modifier

    Issue #6916: migrate tests to j… (compare)

  • Dec 07 18:05

    pbludov on issue-6916-checks-modifier

    Issue #6916: migrate tests to j… (compare)

  • Dec 07 17:53

    pbludov on issue-6916-checks-blocks

    Issue #7330: remove skip_commit… Issue #7330: return exit code f… infra: remove checkout to custo… and 1 more (compare)

  • Dec 07 17:32

    pbludov on issue-6916-checks-metrics

    (compare)

  • Dec 07 17:32

    pbludov on issue-6916-ant

    (compare)

  • Dec 07 17:32

    pbludov on issue-6916-misc

    (compare)

  • Dec 07 17:31

    pbludov on jacoco-660

    (compare)

  • Dec 07 16:21

    romani on issue-6916-checks-blocks

    infra: move spellchecker to one… minor: remove obsolete jacoco s… Issue #7322: add no-error-sprin… and 8 more (compare)

  • Dec 07 08:42

    pbludov on jacoco-660

    Issue #7306: fix RequireThis fo… minor: remove obsolete jacoco s… (compare)

  • Dec 07 08:41

    pbludov on jacoco-660

    minor: remove obsolete jacoco s… (compare)

  • Dec 07 08:14

    pbludov on master

    minor: remove obsolete pmd supp… (compare)

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