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

Tab-autocomplete ruff config #4551

Open
janosh opened this issue May 20, 2023 · 4 comments · May be fixed by #15603
Open

Tab-autocomplete ruff config #4551

janosh opened this issue May 20, 2023 · 4 comments · May be fixed by #15603
Assignees
Labels
cli Related to the command-line interface good first issue Good for newcomers

Comments

@janosh
Copy link

janosh commented May 20, 2023

Would be great if ruff config tab autocompletes:

ruff config is
# tab
ruff config isort

and

ruff config isort.spl
# tab
ruff config isort.split-on-trailing-comma

Related: #2808 hence pinging @not-my-profile

@charliermarsh charliermarsh added the cli Related to the command-line interface label May 22, 2023
@charliermarsh charliermarsh added the good first issue Good for newcomers label Jun 26, 2023
@MichalKacprzak99
Copy link

@charliermarsh Hi, I would like to work on this, can you assign it to me?

@charliermarsh
Copy link
Member

You got it!

@dhruvmanila
Copy link
Member

@MichalKacprzak99 Not in a hurry but just wondering if you've started working on this? Feel free to ask any questions if you're stuck :)

@hesampakdaman
Copy link

hesampakdaman commented Jun 3, 2024

I looked at this issue a little bit during the weekend. I initially tried to create a derived macro for Options that creates an enum that I then implement clap::ValueEnum for. However, this approach is not viable because Rust doesn't support reflection, and I couldn't generate the enum variants for attributes #[option_group] recursively.

My only other idea is to create a huge enum by hand, but that would be a rather big code change.

mishamsk added a commit to mishamsk/ruff that referenced this issue Jan 20, 2025
@mishamsk mishamsk linked a pull request Jan 20, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli Related to the command-line interface good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants