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

* lisp/magit-section.el: Allow several functions as arguments #5112

Closed

Conversation

DamienCassou
Copy link
Contributor


The function magit-disable-section-inserter is convenient in a
.dir-locals.el file to disable a section per repository. When more
than one section should be disabled, this function can be called
several times. This commit makes disabling several sections easier by
allowing more than one section as argument.

The function `magit-disable-section-inserter` is convenient in a
.dir-locals.el file to disable a section per repository. When more
than one section should be disabled, this function can be called
several times. This commit makes disabling several sections easier by
allowing more than one section as argument.
@DamienCassou DamienCassou force-pushed the allow-disabling-several-sections branch from b7f2dae to 0ee0d4b Compare March 18, 2024 07:58
@tarsius
Copy link
Member

tarsius commented Mar 18, 2024

I'm not convinced this is easier. Copying one line and adjusting the new copy, isn't much more involved than moving to the correct place in the only line and then pasting/typing there.

magit-disable-section-inserter can be though of as a variation of remove-hook, which also has to be called once per hook function.

@DamienCassou
Copy link
Contributor Author

Ok for me. Feel free to close the issue.

@tarsius
Copy link
Member

tarsius commented Mar 20, 2024

I might come back to this at some point, but for now I am closing.

@tarsius tarsius closed this Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants