Sorry for the late reply, premises here @SF are largely vacated. The project moved to Github https://github.com/checkstyle/eclipse-cs for project services (Git, Issues) and Bintray for binary downloads. Update site archives starting from version 8.7.0 can be found here: https://bintray.com/eclipse-cs/eclipse-cs/update-site-archive#files/update-site-archive
@Ashis please attach your checkstyle configuration file
FYI the move of eclipse-cs sources to Github as a first measure is a go, see https://github.com/checkstyle/eclipse-cs Thanks to @Bananeweizen for suggesting and to @Roman for taking it under the wings of the Checkstyle organisation at Github
I imagine sevntu metadata is maintained alongside the checks, so whenever you change something (add a property, new check etc.) you will update the eclipse-cs metadata accordingly? For eclipse-cs this is not the case. Instead once in a while I need to check all changes in Checkstyle for their relevance to the metadata. For example: a new check is added a check gets a new property an existing property is changed - for instance changes to allowed or default tokens addition of new message keys * or...
@Roman - thanks and agreed, hosting eclipse-cs sources under the Checkstyle organisation in Github would be the best course. I'll send you a private message shortly. @Richard - interesting, although I am unsure how this can help with eclipse-cs metadata maintenance. As I understand I would still need to meticulously check all changes in a Checkstyle release, whether they have any relevance for the metadata. Ideally Checkstyle itself would expose this metadata (e.g. via annotations) to integration...
Hey Eric. There is effort required for actually updating the plugin to the latest Checkstyle version. Most of the time the bulk of work is hunting down Checkstyle changes which require metadata changes using the Checkstyle release notes. The build itself is automated via Maven/Tycho, however, getting a release out is a multi-stepped, non-automated process. For instance updating the plugin from 8.0 to 8.5 took me about a day of work for updating and releasing. If I had updated to all intermediary...
Oh, how did that slip through... Fixed now and new version 8.5.1 made available.
Configuration via dialog no longer possible with version 8.5
Prepare 8.5.1 release
#429 Configuration via dialog no longer possible with version 8.5
If this is still an issue for you, please report this to the Checkstyle core project, over at https://github.com/checkstyle/checkstyle
eclipse-cs 8.5.0 based on Checkstyle 8.5 has been released.
eclipse-cs 8.5.0 based on Checkstyle 8.5 has been released.
Site fix
Disable jgit build timestamp since it appears not to be working correctly - different timestamps for the subprojects
Did you also manage to get rid of the warning I mentioned? Ah, yes I did, see https://sourceforge.net/p/eclipse-cs/bugs/428/ :-)
ClassCastException: net.sf.eclipsecs.ui.actions.PurgeCachesAction cannot be cast to org.eclipse.core.commands.IHandler
Latest changes are in this branch: https://sourceforge.net/p/eclipse-cs/git/ci/adopt_checkstyle_8.5/tree/ With your additional information I was able to reproduce and fix the issue.
#420 Fix ClasscastException with Quick access bar
#427 Clean Checkstyle markers on Clean-Action
Prepare 8.5.0 release
Use Checkstyle 8.5
#424 Fix for potential NPE
ClassCastException: net.sf.eclipsecs.ui.actions.PurgeCachesAction cannot be cast to org.eclipse.core.commands.IHandler
I don't seem to hit this error with Oxygen.1a and the latest plugin code. Please try 8.5.0 once it's out.
Hey there! You might have noticed - or even been disappointed - by the slow rate of recent updates to this project. After nearly 13 years of maintaining eclipse-cs I noticed that I am losing interest and updating this project is becoming a chore. Also I have been taking up different interest in my spare time, like playing guitar or getting a bit back in shape - which leaves even less time. If you're interested in keeping this project afloat and more regularly updated - and want to lend a hand - please...
Hey there! You might have noticed - or even been disappointed - by the slow rate of recent updates to this project. After nearly 13 years of maintaining eclipse-cs I noticed that I am losing interest and updating this project is becoming a chore. Also I have taking up different interest in my spare time, like playing guitar or getting a bit back in shape - which leaves even less time. If you're interested in keeping this project afloat and more regularly updated - and want to lend a hand - please...
Support CustomImportOrder in eclipse formatter.
Unhandled exception on every file that gets opened
Closing for housekeeping.
NPE on SuppressWarningsHolder
Closing for housekeeping, this issue had been fixed in Checkstyle core (quite) a while back.
Closing for housekeeping, please take it up with Checkstyle core, if this is still an issue for you.
WriteTag warnings do not work when severity is ignore
If this would be done (it could, presumably), how would one discern those exceptions from honest bugs in Checkstyle core? Back in the day, eclipse-cs used to pop up an error dialog, which was truly disturbing. Nowadays it just logs an Checkstyle error, like "hey, something unexpected happened, please check the error log if you care" - which I think is pretty unobtrusive, while still retaining all the information needed for bug hunting. How would you propose to solve this?
'Purge Checkstyle caches' toolbar action violates Eclipse UX standards
This had been improved upon in release 7.6.0 already, sorry for not closing the bug. The toolbar item should now be consistently hidden to start with. Users asking for this feature can un-hide the item by configuring the perspective actionsets accordingly. The button remains ugly, however, but at least the tooltip is more informative.
java.lang.NullPointerException when trying to run checkstyle
I couldn't reproduce the issue with your configuration file, however, I put some safeguards in place to prevent the NPE at the affected position. Fix will be in upcoming 8.5.0
I couldn't reproduce the issue with you configuration file, however, I put some safeguards in place to prevent the NPE at the affected position. Fix will be in upcoming 8.5.0
Error initializing TreeWalker
@Nonnull annotation before ellipsis operator causes exception
This is a problem in Checkstyle core, which might have been fixed already. Please try eclipse-cs 8.5.0 once it's out. If the problem persists please report this to the Checkstyle core project (https://github.com/checkstyle/checkstyle).
Project / Clean operation doesn't remove checkstyle warnings
Project / Clean operation doesn't remove checkstyle warnings
Will be fixed with upcoming 8.5.0
Warnings in Eclipse Console when starting RCP application due to missing category for commands
Fixed for upcoming 8.5.0 release
I am hoping the find the time for the update during my christmas vacation. Apart from that patches/pull requests are certainly welcome. The bulk of work for the upgrade lies in updating the XML metadata for Checkstyle modules the plugin uses to provide the configuration editor.
Apart from that patches/pull requests are certainly welcome. The bulk of work for the upgrade lies in updating the XML metadata for Checkstyle modules the plugin uses to provide the configuration editor.
I'm hoping to find the time for an update during my christmas vacation. Beyond that patches/pull requests are certainly welcome. The bulk of work for the upgrade lies in updating the XML metadata for Checkstyle modules the plugin uses to provide the configuration editor.
Unfortunately I don't think a workaround in eclipse-cs is feasible (or even possible). In my eyes the current classpath scanning implementation offers no practical benefit over the long standing previous implementation while adding serious drawbacks. I hope the points brought forward are sufficient to convince the Checkstyle team to revert this change.
Created https://github.com/checkstyle/checkstyle/issues/4916 - it appears as if a classpath scanning feature introduce in Checkstyle core recently cannot possibly work in Eclipse.
For reference I ended up in https://github.com/checkstyle/checkstyle/blob/master/src/main/java/com/puppycrawl/tools/checkstyle/utils/ModuleReflectionUtils.java#L55 which does not pick up the custom check classes in Eclipse.
Yes, indeed the error message was misleading. I did half an hour of debugging today and from what I can deduce there is something off with the classpath scanning for check class, introduced with https://github.com/checkstyle/checkstyle/issues/3607 However, I couldn't get to the bottom of this just yet - will need more time
I create https://github.com/checkstyle/checkstyle/issues/4885
Possibly this change to PackageNamesLoader: https://github.com/checkstyle/checkstyle/commit/ddeb553834f10570089a033fa1ac1501b74ac358
Hi, I assume this to be related to a Checkstyle-internal change concerning the handling of checkstyle_packages.xml files. Please note in the Exception message that there is a dot missing between the package and class name which Checkstyle core constructs from the checkstyle_packages.xml and the check name from the loaded configuration, e.g. com.puppycrawl.tools.checkstyle.checks.annotationJsr305Annotations Please check with the Checkstyle core project where this change comes from. If my theory holds...
.@Aditya I couldn't reproduce at first glance, please provide steps - preferably in a bug ticket. Judging from the location of the NPE a full clean/rebuild of your workspace might help.
Version 8.0.0 will show up on the update site shortly, including the mentioned fix for Oxygen.
eclipse-cs-8.0.0 has just been released, using checkstyle-8.0. Hope this solves this issue for you, as there won't be any releases for supporting checkstyle 7.7/7.8
Fix version on web page
Compile fix
Switch to Google Checks for own development, reformatting of code
Fix several deprecations and other warnings
Update Checkstyle metadata from 7.6 to 8.0
Error initializing TreeWalker
Checkstyle's NoWhitespaceBefore check simply does not support the DEC token, see http://checkstyle.sourceforge.net/config_whitespace.html#NoWhitespaceBefore You should remove this token from this checks configuration in your Checkstyle configuration file.
Checkstyle's NoWhitespaceBefore check simply does not support die DEC token, see http://checkstyle.sourceforge.net/config_whitespace.html#NoWhitespaceBefore You should remove this token from this checks configuration in your Checkstyle configuration file.
The eclipse-cs plugin uses this option since when it was introduced many years back, to prevent unnecessary execution of "ignored" modules. Since events with the ignored severity are not reported inside eclipse anywhere it would just be a waste to execute them, hence the option cannot be disabled within the plugin.
Upgrade to Checkstyle 8.0
Thanks again for the patch, Mat!
Does not build against Eclipse Oxygen
Thanks, much appreciated!
Help -> Installation details -> Installed Software HTH, Lars
Hi, due to infrastructural issues @Sourceforge, which gave lots of users trouble...
Hi, due to infrastructural issues @Sourceforge, which gave lots of users trouble...
Corrected Bintray update site url
prepate 7.6.0 release
Adopt Checkstyle 7.6
Offer alternative p2 site than https://sourceforge.net/projects/eclipse-cs/files/updatesite/
Hi Michael, I've moved hosting of the update site to Bintray.com. The update site...
Eclipse-CS plugin 7.3.0 cannot be installed from Marketplace or updatesite
Hey all, I've moved hosting of the update site to Bintray.com. The update site url...
eclipse-cs Plug-in 7.3.0 not installing in neon.2
Hey all, I've moved hosting of the update site to Bintray.com. The update site url...
Install from Eclipse Marketplace is failing
Hey all, I've moved hosting of the update site to Bintray.com. The update site url...
Hey all, I've moved hosting of the update site to Bintray.com. The update site url...
Hey all, I've moved hosting of the update site to Bintray.com. The update site url...
Hey all, I've moved hosting of the update site to Bintray.com. The update site url...
Due to changes in Sourceforges SSL/TLS infrastructure "older" java version miss the...
This button was added to realize a feature request (https://sourceforge.net/p/eclipse-cs/feature-requests/165/)...
Hi Michael, the installation issues might also be related to export restricted encryption...
Hi Michael, I did not take it as a complaint, sorry if I came across that way. The...
Apart from that, introducing another way of dynamically loading jars into an OSGI...
eclipse-cs integrates Checkstyle into the Eclipse runtime environment - which is...
Which JDK version is that exactly?
Prepare 7.3.0 release (Pt. 2)
Duncan, it seems the certificate validation exception is related to a new SSL certificate...