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

Fix issue 61 #68

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

sheldonwjones
Copy link

— make g:indent_guides_autocmds_enabled a buffer variable
— this allows file type auto commands like (autocmd FileType python IndentGuidesEnable)

— make g:indent_guides_autocmds_enabled a buffer variable
— this allows file type auto commands like (autocmd FileType python IndentGuidesEnable)
@nathanaelkane
Copy link
Collaborator

Thanks, will give it a test run shortly.

@Quantisan
Copy link

+1 for this feature

@cyprusglobe
Copy link

+1 for this feature as well

@Ram-Z
Copy link

Ram-Z commented Jan 24, 2015

👍

Any particular reason this has not been merged in yet?

@nathanaelkane
Copy link
Collaborator

Any particular reason this has not been merged in yet?

Unfortunately there was a reason, the feature broke some existing functionality and I haven't had time to look into fixing it.

Copy link
Member

@alerque alerque left a comment

Choose a reason for hiding this comment

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

This PR will need updating before merging. Changing the variable to a buffer local is just fine (and a good change) but the buffer local
variable should default to the value of the global setting variable if not otherwise set. That can be done once, then used later in the script as needed.

@alerque alerque marked this pull request as draft February 21, 2023 13:48
@alerque alerque marked this pull request as ready for review February 21, 2023 18:29
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.

None yet

6 participants