Skip to content
This repository has been archived by the owner on Jan 2, 2023. It is now read-only.

consider moving initialisation to lua #23

Closed
matu3ba opened this issue Apr 8, 2021 · 1 comment
Closed

consider moving initialisation to lua #23

matu3ba opened this issue Apr 8, 2021 · 1 comment

Comments

@matu3ba
Copy link

matu3ba commented Apr 8, 2021

I like the plugin very much. Thanks alot for creating!

Reasons to do:

  1. A pure lua profile would decrease startup time.
  2. The lua startup profiler would be more correct or simpler to get correct with this change. Otherwise one gets more wrong results.
  3. Try to be consistent as to use functionality of lua, if possible.
  4. Having a function to initialize makes separating things for user-specific settings while getting updates much easier (user may implement a different function to customize plugin).

Another related reason (bad practice/historical reasons): nvim-treesitter/module-template#1

Relevant content for change:
https://github.com/nanotee/nvim-lua-guide#using-meta-accessors-1

@matu3ba
Copy link
Author

matu3ba commented Apr 8, 2021

sorry, this is a treesitter template issue/bug. will reopen, once that is fixed in the template.

@matu3ba matu3ba closed this as completed Apr 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant