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

Add ability to provide indent size of current document to formatter #9821

Closed
daedroza opened this issue Mar 6, 2024 · 2 comments
Closed
Labels
A-helix-term Area: Helix term improvements C-enhancement Category: Improvements

Comments

@daedroza
Copy link
Contributor

daedroza commented Mar 6, 2024

At the moment, the formatter's capacity to provide information seems limited. There are some workarounds like '{}' to get file_path and so on which aren't merged.

Instead of this, I would like to propose a standardized set of APIs like command expansions that also provide information about current document to be usable inside args of formatters.

Huge projects like chromium/AOSP have multiple programming style. Having a dynamic way to set these properties during runtime of ":format" command would be super helpful.

@daedroza daedroza added the C-enhancement Category: Improvements label Mar 6, 2024
@kirawi
Copy link
Member

kirawi commented Mar 6, 2024

Will be resolved by #6979

@kirawi kirawi added the A-helix-term Area: Helix term improvements label Mar 9, 2024
@pascalkuthe
Copy link
Member

Will be covered by #10389 or #6979 so closing in favor of those

@pascalkuthe pascalkuthe closed this as not planned Won't fix, can't repro, duplicate, stale Apr 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements C-enhancement Category: Improvements
Projects
None yet
Development

No branches or pull requests

3 participants