Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    rnveach
    @rnveach
    @Singleton06 we have an issue on just that but it requires other pieces in place first
    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