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

Add annotations, update deprecations, enforce consistent style with stylua, improve stuff. #25

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

DrKJeff16
Copy link

DON'T MERGE THIS YET, INCOMPLETE PULL REQUEST AS I NEED YOUR INPUT ON THIS.


My Ideas

I plan to write annotations and documentation for this plugin to better communicate to the end user
how utilities should be used and what they do. Also planning to build upon pending stuff.

I have successfully corrected the use of deprecated code in line with more modern versions of nvim.

In addition, I have updated the README.md to include more installation methods,
and intend to write a more complete overview that'll include instructions and examples.

Finally I have implemented stylua to enforce a consistent style of code to improve code readability.


Your Turn

I need to know what you think, what you want me to do and what should I take into account.
I hope we can work together as I'm legitimately interested in helping you out.

If you need any reference for what I can do with Nvim Lua you can check my custom config as to assess whether I can provide any meaningful assistance.

Have a nice day!

I tried to guess what your preffered style was, so I
configured the `.stylua.toml` file accordingly.

In addition, I added a couple of prototype annotations for
easier understanding of what the code does, but I need your
input on this.
Converted to `vim.api.nvim_set_option_value` as the ones you
were using were deprecated and would lead to warnings.
I find using `... in next, table do ...` more consistent and
better overall for use in for loops. What do you think?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant