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

Multiline arguments fail #298

Open
esdmr opened this issue Feb 14, 2024 · 0 comments
Open

Multiline arguments fail #298

esdmr opened this issue Feb 14, 2024 · 0 comments
Assignees
Labels
Blocked bug Something isn't working

Comments

@esdmr
Copy link
Owner

esdmr commented Feb 14, 2024

While calculating the replacement range, given the current token (which is currently coerced into one line), it assumes that the whole thing is in one line, which causes the range to have a negative column. Unfortunately, the output of completions are also split on that newline, so we cannot actually use the output. Again, this would be a non-issue with #209, but I have not figured it out yet. Regardless, to solve it without a parser, we can consider only the last line of the argument and modify that only, skipping the other lines in the completions.

@esdmr esdmr added the bug Something isn't working label Feb 14, 2024
@esdmr esdmr self-assigned this Mar 19, 2024
@esdmr esdmr added the Blocked label Aug 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked bug Something isn't working
Projects
Status: Backlog
Development

No branches or pull requests

1 participant