Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jun 22 00:33

    pbludov on eclipse-r20

    (compare)

  • Jun 21 07:57

    pbludov on eclipse-r20

    dependency: bump eclipse compil… (compare)

  • Jun 21 07:41

    pbludov on eclipse-r20

    dependency: bump eclipse compil… (compare)

  • Jun 21 07:39

    pbludov on eclipse-r20

    Issue #10074: Cover IT folder b… Issue #10138: test ecj jar afte… dependency: bump eclipse compil… (compare)

  • Jun 21 04:14

    pbludov on eclipse-r20

    dependency: bump eclipse compil… (compare)

  • Jun 21 04:12

    pbludov on eclipse-r20

    infra: clean semaphore cache du… Issue #10138: fix semapore prob… Issue #10077: Parse and extract… and 5 more (compare)

  • Jun 19 04:51

    pbludov on eclipse-r20

    dependency: bump eclipse compil… (compare)

  • Jun 19 04:26

    pbludov on eclipse-r20

    dependency: bump eclipse compil… (compare)

  • Jun 19 04:26

    pbludov on eclipse-r20

    dependency: bump eclipse compil… (compare)

  • Jun 19 04:06

    pbludov on eclipse-r20

    dependency: bump eclipse compil… (compare)

  • Jun 19 03:12

    pbludov on eclipse-r20

    dependency: bump eclipse compil… (compare)

  • Jun 18 11:16

    dependabot[bot] on maven

    dependency: bump slf4j-simple f… (compare)

  • Jun 18 11:16
    dependabot[bot] labeled #7
  • Jun 18 11:16
    dependabot[bot] opened #7
  • Jun 18 06:27

    pbludov on eclipse-r20

    Issue #9142: Migrate to Truth i… Issue #10100: Extract parameter… dependency: bump eclipse compil… (compare)

  • Jun 18 02:14

    pbludov on master

    Issue #9105: False Positive Com… Issue #9941: Fixing AtclauseOrd… minor: fixed TeamCity build vio… and 9 more (compare)

  • Jun 14 08:28
    dependabot[bot] labeled #6
  • Jun 14 08:28
    dependabot[bot] opened #6
  • Jun 14 08:28

    dependabot[bot] on maven

    dependency: bump maven-dependen… (compare)

  • Jun 14 08:27
    dependabot[bot] labeled #5
Ullrich Hafner
@uhafner
Wouldn’t it make more sense if this incremental feature is one abstraction higher and not in the CheckStyle library? I have implemented such a feature in Jenkins warnings plugin and it works quite well for CheckStyle (and SpotBugs, PMD, etc.). I.e. the CI tool selects the baseline report that can be used to compare the current results with. The delta algorithm has as input the reports of the baseline and the current build (independent of the used static analysis tool) and is capable of producing a report that identifies new and outstanding warnings.
And one note to checkstyle/checkstyle#3242: it would be quite valuable for my warnings plugin if I can visualize all used suppressions. So if checkstyle suppresses a warning it would be a nice additional feature if that would be part of some (additional and optional) report (PMD does this…)
Dustin Singleton
@Singleton06
Personally, I think I would prefer that it be part of checkstyle. I think that gives more freedom for other tools to make usage of it. We would like to incorporate it into our maven builds, but wouldn't be opposed to also seeing it in jenkins builds, etc, but it's just not our primary use-case.
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