randunel / vim-javascript

0 stars 2 forks source link

Just to let you know... #1

Open FuzzOli87 opened 8 years ago

FuzzOli87 commented 8 years ago

Hello!

I forked vim-js-indent and "manually" merged your changes(copy-paste) And I will probably submit a pull request. I use yajs.vim which doesn't have built in indention. I'll try and help out because this issue bugs me too but I'm not familiar with Vimscript. However, they seem to be simple functions so I'll try and keep bugs noted.

Thanks for working on this.

randunel commented 8 years ago

Hello.

The state of things is getting ridiculous, really. I counted more than 15 JS indentation versions for vim, none of them working as expected.

Nobody is willing to put in the extra hours to build an indentation functionality that's also covered by tests, not even myself (so far). The likelihood of finding a vim user working with javascript who knows vimscript and is willing to do a test driven development of the indentation platform is slim, at most.

I've yet to find a non-abandoned indentation plugin. I'm not looking for an actively maintained one, just a popular one which accepts PRs from all of us. Ideally, it should also be covered by tests, otherwise it's hard to accept pull requests without manually indenting a couple of your projects just to see what has changed.

I only touched vimscript in this project, hence the bad code (nested ifs) that needs refactoring as soon as it reaches the surface, but I'd be happy to assist a maintainer who's willing to guide my PRs.

Anyway, rant over. Wish you best of luck. I've only implemented the features that I needed (that fit my coding style) in this project, which is chain support, so this repo is probably not for the general public. I hope you can find something useful.