-
Notifications
You must be signed in to change notification settings - Fork 301
Lint errors moving on scroll #839
Comments
Hi @Akin909 , Thanks for the kind words (and your support!). Glad you're on the way to using Oni as your daily editor 😄 And appreciate the detailed issue and screenshots too! The underline markers moving with the scroll is definitely a bug - it seems we're not picking up the scroll events in all the right places (which would trigger updating the error adorners/underlines). This should've been fixed by #805 but it looks like there are still some cases where we're not updating proplery. Seems like another issue that it isn't picking up your
Should definitely be possible. You can try these steps here to use ale.vim from within Oni. As an aside, I'm working on a big refactor of our language services (#819) - that will make it easier to switch to disable the existing language services, or switch to alternate strategies (like use flow for typescript, or an alternate TypeScript language server, etc). LMK how it goes or if you hit any issues there! |
Thanks for the reply, Also thanks for the pointer re installing ale in the oni plugins directory i'll give it a go. Out of curiousity I have some plugins which work in Oni out of the box (like easy motion) without needing to be in the oni plugins dir, when does a plugin need to go in there or is the practice with oni that all plugins should really go in there? |
Sure thing! Thanks for the additional info. I tested a bit and did find an issue with release builds not getting the full benefit of TypeScript language services - TypeScript relies on several libraries to define the environment, and those weren't included in the distribution builds. I'm working on a PR for that right now + test case.
It sounds like you have Oni set up to use your My goal though is to make it easy for newcomers to use both Vim and Oni plugins - so someone new to Oni/modal editing doesn't need to know how to set up an |
…853) * Include TypeScript library .d.ts files * Add test case to verify typescript completion * Bring in autocompletion test
Yes I use vim plug to manage my plugins and deoplete for autocompletion, ale for linting (plus dozens more), at the moment though ale which usually shows lint errors with signs in the signcolumn (or gitgutter which shows git changed lines ) aren't working is there a specific config for allowing the sign column? sorry feel like I'm a little of the original topic. |
There shouldn't be anything in Oni stopping this - I just ran this VimL as a litmus test (from http://vimdoc.sourceforge.net/htmldoc/sign.html)
And I do see the sign show up as expected: I wonder if there is some other interaction occurring with Oni that is preventing this from working? I also tested just And it seemed to work OK on my machine. A few things / questions to dig further:
No worries! Sorry you're hitting these issues. It'll be helpful for others if they search for ale or gitgutter anyway 😄 |
I use quite a few, so I tried a minimal vimrc " Plug Setup ===================== {{{
"This will autoinstall vim plug if not already installed
if empty(glob('~/.vim/autoload/plug.vim'))
silent !curl -fLo ~/.vim/autoload/plug.vim --create-dirs
\ https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim
augroup VimPlug
au!
autocmd VimEnter * PlugInstall --sync | source $MYVIMRC
augroup END
endif
call plug#begin('~/.vim/plugged')
Plug 'airblade/vim-gitgutter'
call plug#end() Which unfortunately still didn't work, I also ran the commands above and that worked so it seems the signcolumn works. Lastly I ran None of my executables were found by Ale which makes me wonder if git can be found by gitgutter although when I run a terminal with my full config it has access to fzf and git for example. |
Ah interesting, that's a good find. It's possible those binaries aren't being found - and given gitgutter isn't working, it seems likely that the Running Also, checking the |
Ah good catch, thanks for testing that out. Yes, depending on how it's run, it might not have the same $PATH variable as in your shell (especially if you run it from the dock or finder). You can set the |
I had a go with the additional paths but the path oni is using didn't update at all, and the default vars given to the additional paths variable seem like they should include |
@bryphe I've had a look at the pre-release version and its soo much better 👍 🎆 thanks !! regarding the path issue i've been looking into that and noted #708 relates to this as well(closed), I think NVIM now correctly sources the relevant rc file and gets the PATH, as in the nvim terminal in oni (release version) I get a different path to I've found this module which hopefully addresses paths not being passed in gui apps on Mac OS (which is where I'm running this sorry for the late info) . I think my initial issue re the error highlighting is resolved and I'm getting correct linting errors now from the tslint and tss plugins so I'm happy to close it thanks for the fix. Re. the $PATH i'd be happy to have a go at looking into this although I currently cant figure out how to make changes then build and install the electron app since if I run it via the CLI it all works fine anyway. PS. also happy to leave it be and stop pestering |
@Akin909 glad to hear it's working better! 👍 Regarding the path issue - I believe you're right, it seems like we aren't actually passing the additional paths down to the Neovim process. This is the code we use to spin up the neovim process:
But we're not actually passing down the additional paths variable, it seems. We have a helper method to do that over here: oni/browser/src/Plugins/Api/Process.ts Line 10 in c4b3ce4
So it might just be if we hook that up, Neovim will get the $PATH that it needs. The module you pointed out is interesting too - maybe that could keep us from needing the |
I can look into this and have a go at replacing the How do I build and create the electron app so I can test it outside of the cli environment? |
That would be awesome! Appreciate the help! We have a link on the Wiki for building locally - Development - let me know if you hit any hurdles |
I've had replaced the call to Regarding |
@bryphe this issue is much improved and I'm thinking i'd like to close this though there are 2 tangential issues which persist
Not sure these should be raised as separate issues or discussed here? |
Closing as I think a lot has changed and this issue is a bit dated |
Firstly thanks for the amazing project!! 👍
I'm really looking forward switching to oni as my daily editor.
At the moment though I having some trouble with the linter plugin, which may be a known bug, but I couldn't find reference to it anywhere.
What the screen shot attempts to illustrate is that when the linter runs on save errors show up but on scrolling the underline markers move on scroll so they end up in the wrong place. Also it does not seem to be reading my local tsconfig from the project root or my the tsconfig I have in my home directory.
I've tried with and without the separate linter plugin.
I've also tried deactivating my init.vim config and using only the default config but this error still occurs
Dev tools screenshot
I use ale.vim for linting in iTerm and I wonder if it's possible to use this instead if its something thats still in flux, at the moment the plugin doesn't seem to work
The text was updated successfully, but these errors were encountered: