Age | Commit message (Collapse) | Author |
|
|
|
This corresponds better with the terminology used at:
https://doc.rust-lang.org/rustc/json.html
|
|
There is already a `rust_ignore_secondary_spans` option, providing more
hiding of errors than anyone likely can desire.
|
|
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.
|
|
Add comments referencing documentation.
|
|
(Make cargo work on whole project as documented)
According to `:help ale-integration-rust`, using cargo as a linter should
check the entire project. It appears this was the case initially, but it
broke somewhere along the way.
Setting `filename` to the output in the handler and avoid filtering the
spans on `file_name` goes a long way in restoring the intended behavior.
However test cases fail, even after attempting to update them. Please
see next commit.
|
|
|
|
|
|
|
|
* fix: duplicate tag in doc
* fix: duplicate tags in doc
Co-authored-by: Sébastien NOBILI <code@pipoprods.org>
|
|
* Dispatch textDocument/didChange after rename
Previously whenever we renamed a symbol that was referenced from other
files we'd just edit those files in the background, and the LSP wouldn't
know about these changes. If we tried to rename the same symbol again,
the renaming would fail. In some scenarios, the operation would just be
wrong. Here is an attempt to fix this issue.
I also noticed another bug when using Go with `gopls` LSP and the `gofmt`
fixer. Whenever the file was saved, the `gofmt` would run and reformat
the file. But it seems there was some kind of a race condition so I
disabled saving for now, and all of the modified files will be unsaved,
so the user should call `:wa` to save them. I personally like this even
better because I can inspect exactly what changes happened, and I
instantly see them in the other opened buffers, which was previously not
the case.
Fixes #3343, #3642, #3781.
* Address PR comments
* Remove mode tests in corner case tests
* Address PR comments
* Save after ALERename and ALEOrganizeImports
Also provide options to disable automatic saving, as well as instructions to
enable `set hidden` before doing that.
* Fix broken test
* Save only when !&hidden
* Update doc
* Update doc
* Add silent
|
|
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.
|
|
The Black project moved to the PSF organization some time ago. Update
its URL accordingly.
|
|
|
|
* 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
|
|
|
|
* Allows to use quickfix for references.
E.g. following mapping could be used to find references for item under
cursor and put result into quickfix list:
```
nnoremap <leader>af :ALEFindReferences -quickfix<CR>
```
Fixes #1759
* Documentation update.
|
|
Only open list window if the number of warnings or errors equals to or
exceeds the value of ale_open_list. No change when set to `1`.
Co-authored-by: cos <cos>
|
|
Closes #1810
Add ALEPopulateQuickfix and ALEPopulateLocList. They're not very useful
with ale's default auto-populate behaviour, so their useful configuration
is described in help.
|
|
|
|
* Add WGSL support using `naga` command
* Add documents for wgsl
* Add test for `naga` linter
* Separate naga handler callback to hanlder/naga.vim
|
|
* enable using cpplint for c
* fix tag alignment
* fix tag alignment trial #2
Co-authored-by: Justin Huang <justin.huang@perceive.io>
|
|
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>
|
|
* Fix 4004 - Disable eslint by default for json.
This PR disables, or more correctly, excludes eslint from the list of
default linters for json files.
Also fixes elixir, go, json5, and jsonc files documentation and default
linters to make them consistent.
* Fix and improve tests
|
|
* Fix 4025 - Allow to configure texlab LSP settings
* Fix ale_tex_texlab_config type in help
|
|
|
|
* Add dprint fixer
* Fix windows tests
* dd dprint documentation
|
|
|
|
|
|
* ALEFileRename command added.
This command renames file and uses tsserver `getEditsForFileRename` to
fix import paths in Typescript files.
* ale#util#Input fix
* Even more fixes.
* Linting error fix.
|
|
When `let g:ale_ruby_reek_show_wiki_link = 1`, Reek linter is crashed
because `wiki_link` attribute does not exist in result.
It seems that `wiki_link` is now replaced with `documentation_link` in
recent version of Reek
|
|
* 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
|
|
* Pass full path to pydocstyle
* Update pydocstyle tests
|
|
|
|
If virtualtext.vim is autoloaded first, it will link
ALEVirtualTextWarning to ALEWarning. But ALEWarning is not initialized
yet, so it will create ALEWarning, but with no color definition set.
Shortly after, highlight.vim is autoloaded, which would usually link
ALEWarning to SpellCap, but only if ALEWarning is not already set.
However since ALEWarning is already initialized due to the previous
link, we skip this and never actually come around to properly
initializing it.
We fix this by initializing all highlight groups in highlight.vim, thus
satisfying the dependency of ALEVirtualTextWarning being initialized
after ALEWarning.
Fixes #3585
|
|
* feat-draft: inital volar setup
* feat(volar): add documentation
* feat(volar): include default init opts
* feat(volar): add initial tests
* fix(volar): add possible project roots
* fix(volar): tests - use empty files
|
|
* Add cppcheck handler match on misra msg
* Fix cppcheck --file-filter setting
This time, the tests and actually usage both work.
Co-authored-by: Dan George <dgeorge@anduril.com>
|
|
* 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>
|
|
* Add cppcheck handler match on misra msg
* Use --file-filter cppcheck option
Cppcheck recently added --file-filter so that cppcheck only checks the
filtered files, even when using --project option, which checks all files
in the project, by default. The --ccpcheck-build-dir option didn't help
enough (at all?).
* Added C test cases
Also fixed and assumed typo: foo.c, instead of foo.cpp
* Replace hard-coded full path filenames
Attempt to fix the windows platform test execution.
* Fix typo - foo.c, instead of foo.cpp
* Reset buffer var between tests
* Handle header files in cppcheck
Cppcheck isn't designed to check header files, stand-alone. Daniel
Marjamäki suggested using --suppress options to avoid FPs.
* Fix Vint complaint in cppcheck handler.
* Fix file path in cppcheck handler
Co-authored-by: Dan George <dgeorge@anduril.com>
|
|
* Add cppcheck handler match on misra msg
* Add linebreak to checkmake formatter
Co-authored-by: Dan George <dgeorge@anduril.com>
|
|
Since having been added, the `alex` tool has added support for linting
on stdin. Rewrite this integration to reduce the number of tools
requiring disk-write access.
Signed-off-by: David Houston <houstdav000@gmail.com>
|
|
* Adds phpactor lsp linter
* Fixes missing comma
* Adds tests for phpactor lsp linter
* Adds note that this part is not my own work
* Removes unused variable
* Adds phpactor to supported tools list
* Fixes doc sorting
* Wraps phpactor in code tags
|
|
* Look for nearest composer.json before .git
* Add test for projects with composer.json
|
|
* 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.
|