Age | Commit message (Collapse) | Author | |
---|---|---|---|
2018-10-02 | Fix PMD not working with classes without package | Diego Lemos | |
PMD is currently not working properly for Java classes that use [unnamed packages](https://docs.oracle.com/javase/specs/jls/se11/html/jls-7.html#jls-7.4.2). Consider the following Java class that does not contain a `package` declaration: ```java public class App { String getGreeting() { return "Hello world."; } static void main(String... args) { System.out.println(new App().getGreeting()); } } ``` Running PMD in the command line agaist the Java class above produces an output with empty string `""` in the `"Package"` column: ```sh $ pmd -R category/java/bestpractices.xml -f csv -d './src/main/java/App.java' Oct 02, 2018 9:10:39 PM net.sourceforge.pmd.PMD processFiles WARNING: This analysis could be faster, please consider using Incremental Analysis: https://pmd.github.io/pmd-6.7.0/pmd_userdocs_incremental_analysis.html "Problem","Package","File","Priority","Line","Description","Rule set","Rule" "1","","/Users/diego/Projects/github.com/dlresende/kata-fizz-buzz/src/main/java/App.java","2","7","System.out.println is used","Best Practices","SystemPrintln" ``` But the pmd.vim handler's current pattern refuses everything coming from a Java class that does not have a package name (2nd column): ```vim let l:pattern = '"\(\d\+\)",".\+","\(.\+\)","\(\d\+\)","\(\d\+\)","\(.\+\)","\(.\+\)","\(.\+\)"$' ``` The solution I am proposing is to also accept empty strings as package names. | |||
2018-09-04 | Improve ALE project style checking | w0rp | |
* The project style linter now runs while you type. * Now the scripts for checking the project require blank lines. * Many style issues have been found and fixed. | |||
2018-08-26 | Add vader test and fix command callback. | Horacio Sanson | |
2018-08-24 | Add vscode-java-language-server linter | Horacio Sanson | |
2018-08-02 | Simplify the code for most linters and tests with closures | w0rp | |
2018-07-15 | Massively reduce the amount of code needed for linter tests | w0rp | |
2018-04-09 | Add support for the java PMD linter | Johannes Wienke | |
2018-04-08 | Close #1476 - Make the javac executable configurable | w0rp | |
2017-11-20 | Fix #859 Include test and jaxb Java source paths when available | w0rp | |
2017-11-18 | #852 - Capture error codes for checkstyle | w0rp | |
2017-11-05 | Fix #1061 - Handle the filenames returned by javac | w0rp | |
2017-08-29 | Gradle support for javac | Brayden Banks | |
Based off of #745. | |||
2017-08-08 | Ban use of ==# or ==? in the codebase, and prefer is# or is? instead | w0rp | |
2017-06-19 | handle column number in javac linter (#660) | oaue | |
* handle column number in javac linter * Updated tests with column number for javac errors. * Updated tests with column number for javac errors. | |||
2017-05-28 | Revert "Fix #501 - Do not run javac when it is just a stub asking you to ↵ | w0rp | |
install Java on Mac OSX" This reverts commit 528355e2c6cf64fbc0d459a7a512a7823f90043d. | |||
2017-05-15 | Add checkstyle linter | Devon Meunier | |
2017-05-12 | #549 Temporarily revert shell escaping changes, just for Windows | w0rp | |
2017-05-08 | #540 Fix shell escaping pretty much everywhere | w0rp | |
2017-05-04 | #502 Parse more undefined symbol errors | w0rp | |
2017-05-04 | Fix #502 - Report undefined symbol errors better for javac | w0rp | |
2017-05-04 | Fix #501 - Do not run javac when it is just a stub asking you to install ↵ | w0rp | |
Java on Mac OSX | |||
2017-04-25 | Fixes #361, fixes #417 Get classpaths from Maven, and automatically detect ↵ | w0rp | |
src/main/java paths | |||
2017-04-18 | Add a function for getting matches, and use it to simplify a lot of code | w0rp | |
2017-04-16 | #427 Implement buffer variable overrides for all linter options | w0rp | |
2017-03-31 | Fix #438 Create Java .class files for javac in a temporary directory | w0rp | |
2017-03-30 | Remove 'col' from linters where it is hardcoded to 1 (#434) | Adriaan Zonnenberg | |
* Remove 'col' from linters where it is hardcoded to 1 When 'col' is 1, the first column will get highlighted for no reason. It should be 0 (which is the default). In the scalac linter there was also a check about the outcome of `stridx`. It would set l:col to 0 if it was -1, and then it uses `'col': l:col + 1` to convert the outcome of `stridx` to the actual column number. This will make 'col' equals 1 when there is no match. We can remove the check because `-1 + 1 = 0`. * Remove outdated comments about vcol vcol was added as a default, and the loclists that follow these comments do not contain 'vcol' anymore | |||
2017-02-26 | Copy all loclist items returned from handlers, and set up defaults for ↵ | w0rp | |
convenience | |||
2017-02-11 | Fix the custom check issue. | w0rp | |
2017-02-11 | Make javac work in a basic way | w0rp | |
2017-02-11 | Added support for javac (with eclipse classpath support for now) (#141) | Valentin Finini | |
* A try at javac support for ALE * Small cleanup: moved '/tmp/java_ale' string into script var * Fixed Travis-CI build failing on autocmd not being in augroup and stupid omission * One more fix for Travis-CI * For some reason, expandtab was not set * Indentation and removal of header guard. Used examples from ale_linters/c/gcc.vim and ale_linters/javascript/eslint.vim for the indentation of string concat blocks. |