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

speed up setup time by conditionally loading nvim-treesitter.query and nvim-treesitter.indent #213

Merged
merged 1 commit into from
Sep 12, 2021

Conversation

goolord
Copy link
Contributor

@goolord goolord commented Sep 9, 2021

according to PackerProfile this change makes setup go from taking ~22ms to ~0.4ms on my machine. unsure why requiring these files is so slow.

@goolord
Copy link
Contributor Author

goolord commented Sep 9, 2021

the effect is less dramatic without using https://github.com/lewis6991/impatient.nvim
(~6ms -> ~2ms), so I'll report it there as well

Copy link
Owner

@lukas-reineke lukas-reineke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good 👍

just a couple small changes

lua/indent_blankline/init.lua Outdated Show resolved Hide resolved
lua/indent_blankline/init.lua Outdated Show resolved Hide resolved
lua/indent_blankline/init.lua Outdated Show resolved Hide resolved
lua/indent_blankline/init.lua Outdated Show resolved Hide resolved
lukas-reineke
lukas-reineke previously approved these changes Sep 10, 2021
Copy link
Owner

@lukas-reineke lukas-reineke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@lukas-reineke lukas-reineke changed the base branch from master to develop September 10, 2021 22:52
@lukas-reineke lukas-reineke dismissed their stale review September 10, 2021 22:52

The base branch was changed.

@lukas-reineke
Copy link
Owner

oh sorry, can you please rebase against develop and fix the conflictc

lukas-reineke
lukas-reineke previously approved these changes Sep 10, 2021
@lukas-reineke
Copy link
Owner

@goolord you need to rebase, not merge in develop.
I want the project to have a linear history.
You need to run git rebase develop

Copy link
Owner

@lukas-reineke lukas-reineke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@lukas-reineke lukas-reineke merged commit 587839c into lukas-reineke:develop Sep 12, 2021
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.

2 participants