Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The plugin should report failure when it cannot import/autocomplete due to syntax errors. #367

Closed
ym1234 opened this issue Jan 31, 2018 · 5 comments
Labels

Comments

@ym1234
Copy link

@ym1234 ym1234 commented Jan 31, 2018

If I open a file with incorrect syntax, and try to import missing imports, the plugin silently fails instead of reporting the error/warning/notice.

Actual behavior (Required!)

Silently fails.

Expected behavior (Required!)

Report an error/warning/notice.

The steps to reproduce actual behavior (Required!)

  1. Open an file with syntax error.
  2. Try to import missing imports.

Environment (Required!)

  • OS: Arch Linux
  • Vim version:
  • Neovim version: 0.2.2

Q&A

  • Yes, I tried minimal .vimrc configuraion.
  • Yes, I have enabled logs with JCdebugEnableLogs and can put here content of JCdebugGetLogContent command, if you need.
  • Even, if you wish, I can set g:JavaComplete_JavaviDebug to 1, than set g:JavaComplete_JavaviLogfileDirectory, and put here server logs, too.
@artur-shaik

This comment has been minimized.

Copy link
Owner

@artur-shaik artur-shaik commented Jan 31, 2018

Year, I agree. We should inform about this.

@livingBEEF

This comment has been minimized.

Copy link

@livingBEEF livingBEEF commented Mar 8, 2018

It would be nice if it could populate location-list too. The ale plugin can already do that, but

  • the compilation has to be done twice - once for ale and once for javacomplete2
  • ale's support for gradle seems buggier
@wsdjeg

This comment has been minimized.

Copy link
Contributor

@wsdjeg wsdjeg commented Mar 8, 2018

why localtion list is need, what info do you want to shown in it, and what should it done when you press Enter or run :lnext?

@livingBEEF

This comment has been minimized.

Copy link

@livingBEEF livingBEEF commented Mar 8, 2018

My idea was something like error hinting that IDEs usually do, so it would populate location list with encountered errors (possibly also highlight the errors, if it's not too difficult). Another question is, whether it should leave it to the users how they use the location-list, or whether it should add its own line markings and such, but either would be fine with me.

@wsdjeg

This comment has been minimized.

Copy link
Contributor

@wsdjeg wsdjeg commented Mar 8, 2018

@livingBEEF Javacomplete is omni plugin for java, I do not think it should provides lint featurem you can use neomake which should support maven and grandle project. and the neomake's java lint is maintained by me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.