diff options
| author | Sander van Harmelen <sander@xanzy.io> | 2017-05-20 12:43:28 +0200 |
|---|---|---|
| committer | w0rp <w0rp@users.noreply.github.com> | 2017-05-20 11:43:28 +0100 |
| commit | 455793dfd9c657a76d7a143e0e0eb395d779379b (patch) | |
| tree | 1ca96c5fae697720e76a646e7f20b13c15a0c76e /test/handler | |
| parent | bb1f045d59e93b0565927f1e887e3e7a3a11e61b (diff) | |
| download | ale-455793dfd9c657a76d7a143e0e0eb395d779379b.zip | |
Improve performance when using gometalinter (#566)
* Improve performance when using gometalinter
Before this change when I opened a big project that had 6000+ warnings/errors it took ages to get the actual warnings/errors and it caused my CPU to be busy for quite some time. The call to gometalinter alone took about 24 seconds, but after that vim was struggling as well.
After this change the gometalinter call just takes 2 seconds and nothing noticable happens with the CPU and/or vim.
* Removed obsolete test
This logic is no longer done by the `ale` plugin, but by `gometalinter` itself.
Diffstat (limited to 'test/handler')
| -rw-r--r-- | test/handler/test_gometalinter_handler.vader | 21 |
1 files changed, 0 insertions, 21 deletions
diff --git a/test/handler/test_gometalinter_handler.vader b/test/handler/test_gometalinter_handler.vader index 52a4fc96..603ba22d 100644 --- a/test/handler/test_gometalinter_handler.vader +++ b/test/handler/test_gometalinter_handler.vader @@ -51,24 +51,3 @@ Execute (The gometalinter handler should handle relative paths correctly): \ 'baz.go:12:3:warning: expected ''package'', found ''IDENT'' gibberish (staticcheck)', \ 'baz.go:37:5:error: expected ''package'', found ''IDENT'' gibberish (golint)', \ ]) - - -Execute (The gometalinter handler should filter out errors from other files): - silent file! /some/path/sql.go - - AssertEqual - \ [], - \ ale_linters#go#gometalinter#Handler(bufnr(''), [ - \ '/some/path/interface_implementation_test.go:417::warning: cyclomatic complexity 24 of function testGetUserHeaders() is high (> 10) (gocyclo)', - \ '/some/path/sql_helpers.go:38::warning: cyclomatic complexity 11 of function CreateTestUserMetadataDB() is high (> 10) (gocyclo)', - \ '/some/path/sql_alpha.go:560:7:warning: ineffectual assignment to err (ineffassign)', - \ '/some/path/sql_alpha.go:589:7:warning: ineffectual assignment to err (ineffassign)', - \ '/some/path/sql_test.go:124:9:warning: should not use basic type untyped string as key in context.WithValue (golint)', - \ '/some/path/interface_implementation_test.go:640::warning: declaration of "cfg" shadows declaration at sql_test.go:21 (vetshadow)', - \ '/some/path/sql_helpers.go:55::warning: declaration of "err" shadows declaration at sql_helpers.go:48 (vetshadow)', - \ '/some/path/sql_helpers.go:91::warning: declaration of "err" shadows declaration at sql_helpers.go:48 (vetshadow)', - \ '/some/path/sql_helpers.go:108::warning: declaration of "err" shadows declaration at sql_helpers.go:48 (vetshadow)', - \ '/some/path/user_metadata_db.go:149::warning: declaration of "err" shadows declaration at user_metadata_db.go:140 (vetshadow)', - \ '/some/path/user_metadata_db.go:188::warning: declaration of "err" shadows declaration at user_metadata_db.go:179 (vetshadow)', - \ '/some/path/queries_alpha.go:62::warning: Potential hardcoded credentials,HIGH,LOW (gas)', - \ ]) |