Skip to content

Fix issue 61 - #68

Open
sheldonwjones wants to merge 1 commit into
preservim:masterfrom
sheldonwjones:master
Open

Fix issue 61#68
sheldonwjones wants to merge 1 commit into
preservim:masterfrom
sheldonwjones:master

Conversation

@sheldonwjones

Copy link
Copy Markdown

— 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
Copy Markdown
Collaborator

Thanks, will give it a test run shortly.

@Quantisan

Copy link
Copy Markdown

+1 for this feature

@cyprusglobe

Copy link
Copy Markdown

+1 for this feature as well

@Ram-Z

Ram-Z commented Jan 24, 2015

Copy link
Copy Markdown

👍

Any particular reason this has not been merged in yet?

@nathanaelkane

Copy link
Copy Markdown
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.

@alerque alerque left a comment

Copy link
Copy Markdown
Member

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.

6 participants