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

Alt-a should support --parameters with double dashes #3992

Closed
1 task done
Jaykul opened this issue Apr 9, 2024 · 0 comments · Fixed by #4016
Closed
1 task done

Alt-a should support --parameters with double dashes #3992

Jaykul opened this issue Apr 9, 2024 · 0 comments · Fixed by #4016
Labels

Comments

@Jaykul
Copy link

Jaykul commented Apr 9, 2024

Prerequisites

  • Write a descriptive title.

Description of the new feature/enhancement

The Alt+A key that navigates between parameters doesn't properly handle parameters from posix-friendly apps like the azure cli and kubectl that use two dashes instead of one. I. E. --output "json" instead of skipping the parameter name, it selects it.

Proposed technical implementation details (optional)

We could argue this is a parser bug, because the root cause is probably that the AST doesn't represent --output json as a parameter name and value, but instead as two string arguments.

However, you could solve it without changing the AST, by just checking for the -- on the front of the value...

@Jaykul Jaykul added the Issue-Enhancement It's a feature request. label Apr 9, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Triage 🔍 It's a new issue that core contributor team needs to triage. label Apr 9, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot removed the Needs-Triage 🔍 It's a new issue that core contributor team needs to triage. label May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants