Age | Commit message (Collapse) | Author |
|
Please see previous commit for the change requiring test updates.
Tests fail due to mismatch between filename being set to '<anon>' and
the proper filename. That's not a surprise, as the code path setting the
value to anything but the actual filename has been disabled.
It is unclear to me what '<anon>' is supposed to indicate, or how it
should be used. Searching for the string in ale's codebase gives no
matches outside of the rust handler and test cases.
There are also failures due to finding the wrong span. Figuring out how
to fix the '<anon>'/filename mismatch will likely fix that one too.
|
|
Unimport (https://github.com/hakancelik96/unimport/) is a linter,
formatter for finding and removing unused import statements.
This introduces linting support, although fixer support could come
later.
|
|
* Add CMake linter cmake-lint
Add support for the CMake linter provided by
https://github.com/cheshirekow/cmake_format.
* Escape cmake-lint executable and add linter tests
|
|
|
|
* Add WGSL support using `naga` command
* Add documents for wgsl
* Add test for `naga` linter
* Separate naga handler callback to hanlder/naga.vim
|
|
Previously, it would not generate any lint messages for nix 2.5.
Moreover, it would cause this error whenever the nix command is
invoked, when paired with a custom `g:ale_command_wrapper`:
Error detected while processing function <SNR>92_NeoVimCallback[29]..<lambda>27[
1]..<SNR>90_ExitCallback[28]..ale_linters#nix#nix#Command:
line 1:
E684: list index out of range: 0
|
|
Co-authored-by: Horacio Sanson <horacio@allm.inc>
|
|
* add support for checkov for terraform
* add tests for checkov handler
* add basic linter config tests for checkov
* update supported tools and languages lists
* simplify ale_linters#terraform#checkov#Handle
* ensure "-o json --quiet" is always set for checkov
* add documentation for checkov including config options
* fix tests after changing handling of default options for checkov
* add checkov to list of tools in doc/ale.txt
|
|
* Add cspell linter
Add cspell linter, with the languages it supports.
Signed-off-by: David Houston <houstdav000@gmail.com>
* Add cspell Global Variables Documentation
Add documentation to /doc/ale.txt with cspell configuration options.
Signed-off-by: David Houston <houstdav000@gmail.com>
* Add cspell to docs, Minor Cleanup
Add cspell for each supported language, adding some spaces and removing
others when caught navigating the file.
Signed-off-by: David Houston <houstdav000@gmail.com>
|
|
* Show errors and warnings for the 'smlnj' linter
Fixes #3953
* Change smlnj stdIn regex
|
|
Co-authored-by: Dan George <dgeorge@anduril.com>
|
|
* Add support for AVRA linting
* Add tests for AVRA linting and improve code
* Fix test
* Fix warning detection
* Fix test
* Fix test
* Add AVRA as a supported language in docs
|
|
Elvis is Erlang style reviewer, thus all loclist objects should have
sub_type set to style.
|
|
* Add Statix for Linting
Add `statix check` as a linter. Provides a simple set of definition
tests additionally. Variable names specify "check" to allow for later
addition of `statix fix` as a formatter once stream support is added.
Signed-off-by: David Houston <houstdav000@gmail.com>
* Fixup Supported Tools List
I didn't realise there were two separate lists of tools, so add statix
to the other list. Also, remembered "S" comes after "R", and so
re-ordered it.
Signed-off-by: David Houston <houstdav000@gmail.com>
* Fix statix Test File
I refactored the variables for statix to allow for writing a fixer
later, and forgot to update them in the test, so update them now. Also
remove a stray "i", add missing space before checks
Signed-off-by: David Houston <houstdav000@gmail.com>
* Update Output Stream for v0.4.0
statix v0.4.0 provides a breaking change of output stream from stderr to
stdout.
Signed-off-by: David Houston <houstdav000@gmail.com>
* Add statix fix Fixer
Implement statix fix as a fixer for simple Nix antipatterns.
Signed-off-by: David Houston <houstdav000@gmail.com>
* Fix statix Fixer Tests
Fix the statix fixer tests by removing the unnecessary
'read_temporary_file' value from the command, since it simply uses the
default value.
Signed-off-by: David Houston <houstdav000@gmail.com>
* Add statix Handler Test
Add a test for the statix handler per @hsanson's request.
Signed-off-by: David Houston <houstdav000@gmail.com>
* Fix to run only on stdin for linting
Signed-off-by: David Houston <houstdav000@gmail.com>
|
|
|
|
Some custom linter hooks don't include these numbers.
|
|
|
|
* support for erblint
* fix tests
* test for handler
* wrong names
* typo
* doc layout
* CI failed?
|
|
|
|
|
|
* Create rflint.vim
support for robot framework by creating ale definition for rflint syntax linter
* robot framework - rflint support
|
|
More recent versions of thriftcheck use a more compliant GCC-style
output format which includes a space before the "severity" group.
This matches similar tools, like shellcheck.
This change adjusts the handler's pattern to parse this format in a
backwards-compatible way (even though backwards compatibility isn't
critical long-term as thriftcheck itself is close to its 1.0 release).
|
|
* Add support for cppcheck 1.34
* Add cppcheck 1.34 tests, correct pattern
Co-authored-by: Tyler S. Jordan <tsjorda@sandia.gov>
|
|
ThriftCheck (https://github.com/pinterest/thriftcheck) is a linter for
Thrift IDL files.
|
|
|
|
* Add yosys for verilog files.
* Add handler test for yosys.
* fix typo in yosys handler test
* fix array order in yosys handler test
* add yosys linter to filetype defaults test
* fix duplicate tag
* add 'yosys' to 'ale-supported-languages-and-tools.txt'
|
|
This is achieved by switching to JSON, which makes it much easier to
avoid confusion between an error message and the next one. It also
spares us from having to deal with regular expressions, and eliminates
some edge cases that no longer need to be tested.
|
|
- Show hadolint rule number in vim gutter in addition to `ALEDetails`
- Capture and show error in case of syntax errors
- Add tests for error capture
- Adapt existing tests
fixes: #2333
fixes: #958
|
|
|
|
|
|
This allows the location list from one buffer to point to an issue in
another; previously, the error message would be shown but with no way to
jump to it.
|
|
|
|
* fix: added support for local solhint executable
* feat: added support for matching parse errors
* test: added test for solhint command callback and handler
* chore: removed command callback test
* refactor: made solhint handler structure closer to eslint
|
|
|
|
* improve DMD handler
- ignore errors from other files
- catch 'Deprecation' as warning
- add tests
* adding filename key instead of filtering
* update dmd test
* fix test dmd windows
|
|
* v: add "v fmt" fixer.
* v: add "v" (build) linter.
* v: fix vlint complaints and add documentation.
* v: add tests.
* v: use ale#Pad().
|
|
|
|
|
|
|
|
|
|
|
|
* Fix 3605 - set fallback for error detail
* Add use case with no detail key
Co-authored-by: Horacio Sanson <horacio@allm.inc>
|
|
* Fix ansible-lint linter definition.
Use ansible-lint's feature auto-detection instead of temporary file.
For auto-detection to work, ansible project has to be also a git repository.
Don't use yaml rules. These are checked by yamllint.
Refactor pattern to work with ansible-lint >=5.0 version.
Clean-up obsolete test cases.
* Pull Request changes
|
|
* Simplify verilator linter using ale command format strings
* Verilator Linter: Restructure linter command tests
* Verilator Linter: adds to the handler test the returned filename
* Verilator Linter: add the current file path to the search path
* Verilator Linter: Add the search path to the tests
Co-authored-by: TG <tarik.graba@telecom-paris.fr>
|
|
|
|
hadolint: Recognize message type
|
|
|
|
Instead of using `terraform fmt` for linting use `terraform validate`
with json output.
|
|
Update error-parser for `nix-instantiate` on Nix 2.4+
|
|
In the (unreleased) Nix 2.4 the error-messages have been reformatted[1].
This patch aims to retain proper `.nix`-support in `ale`, for both
stable Nix (2.3 and older) and unstable Nix (2.4 and newer).
[1] https://github.com/NixOS/nix/pull/3590
|