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

Better documentation (import sorting) #449

Open
danielniccoli opened this issue Apr 15, 2024 · 1 comment
Open

Better documentation (import sorting) #449

danielniccoli opened this issue Apr 15, 2024 · 1 comment
Labels
documentation Improvements or additions to documentation

Comments

@danielniccoli
Copy link

danielniccoli commented Apr 15, 2024

I've been trying to get import sorting working.

There is one settings-sample that shows this:

grafik

Looking into the default settings, I found this

grafik

The way I first understood it was, that the setting from the first screenshot is already set by default. It took me some research to understand that the first setting by default only allows the setting from the second screenshot to be enabled in the first place. And I still had to explicitly set "source.organizeImports": "explicit".

I find this confusing, because I expected the first table to be a comprehensive list of possible settings. But the table shows only half of what's possible and the samples add the rest. That settings for different stuff have the same name's adds to the confusion. I also found the samples a bit confusing, because it doesn't really explain why a certain configuration is achieved with a certain set of settings.

What I think would improve this, is a comprehensive list of all settings and what they do.

@AlexWaygood AlexWaygood added the documentation Improvements or additions to documentation label Apr 15, 2024
@dhruvmanila
Copy link
Member

I've been trying to get import sorting working.

Can you expand on what you're trying to achieve?

I find this confusing, because I expected the first table to be a comprehensive list of possible settings. But the table shows only half of what's possible and the samples add the rest. That settings for different stuff have the same name's adds to the confusion. I also found the samples a bit confusing, because it doesn't really explain why a certain configuration is achieved with a certain set of settings.

Correct me if I'm wrong but I think there's some confusion here. Assuming that the table you're mentioning is this, the table only lists the VS Code settings that the extension provides while the sample shows how you can configure the VS Code editor for common scenarios. This means that the settings shown in the samples are all part of the editor and not the extension. For a comprehensive list of settings that the editor provides, you might want to refer to their documentation.

I guess the confusion is coming from the fact that both the extension and the editor provides a setting where "organizeImports" is being used while both have a different meaning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants