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

editorconfig-mode-apply and autoload #357

Open
bestlem opened this issue Aug 26, 2024 · 3 comments
Open

editorconfig-mode-apply and autoload #357

bestlem opened this issue Aug 26, 2024 · 3 comments

Comments

@bestlem
Copy link

bestlem commented Aug 26, 2024

I am trying to use editorconfig-mode-apply for Org-src-mode as suggested in #142 (comment)

However it is failing to work in my setup as the function is void.

Looking at the code I see it is not autoloader and then Changelog for 0.11.0 says other functions are autoloaded and this one is not.

Why is this?

@monnier
Copy link
Collaborator

monnier commented Aug 26, 2024

Does org-src-mode obey the .dir-locals.el settings?
If it does not, then I suggest you file a bug report about it to the Org maintainers.
[ And if it does, then the editorconfig bundled with Emacs-30 should "just work". ]

@bestlem
Copy link
Author

bestlem commented Aug 26, 2024

I want to set this for all org-mode not just in a single directory so why is .dir-locals.el relevant?

@monnier
Copy link
Collaborator

monnier commented Aug 26, 2024

IIUC you want to use .editorconfig files to set some options in some org-mode situations, including org-src-mode. IIUC this already works for normal Org files but not org-src-mode.
My question is whether you get the same problem when using .dir-locals.el instead of .editorconfig.

If that's the case, then I'd argue the problem is not in editorconfig but in Org, so you should (also) report it as a bug there.

[ And I also mention that if it works with .dir-locals.el, then this editorconfig problem should be fixed in the upcoming Emacs-30 where the bundled editorconfig support hooks itself a bit differently inside Emacs and should work just as well as .dir-locals.el. ]

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

No branches or pull requests

3 participants
@bestlem @monnier and others