We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
golines will shorten a declaration like this:
foo := longlistfunc(bigarg, otherbigarg, verylongarg, veryverylongarg, veryveryverylongarg, extremelylongarg)
to
foo := longlistfunc( bigarg, otherbigarg, verylongarg, veryverylongarg, veryveryverylongarg, extremelylongarg, )
I would expect it to similarly shorten a declaration like
foo := longlistfunc(bigarg, otherbigarg, verylongarg, veryverylongarg, veryveryverylongarg, extremelylongarg)[0]
foo := longlistfunc( bigarg, otherbigarg, verylongarg, veryverylongarg, veryveryverylongarg, extremelylongarg, )[0]
but it is left alone. This appears to be due to formatExpr not supporting index expressions. It results in a debug message like
Got an expression type that can't be shortened: *dst.IndexExpr
The text was updated successfully, but these errors were encountered:
Also seeing this bug; it's very confusing, and causing issues with the use of the golangci-lint linter lll.
golangci-lint
lll
@yolken-segment - Is there anything preventing a fix for this issue from moving forward?
Sorry, something went wrong.
Successfully merging a pull request may close this issue.
golines will shorten a declaration like this:
to
I would expect it to similarly shorten a declaration like
to
but it is left alone. This appears to be due to formatExpr not supporting index expressions. It results in a debug message like
The text was updated successfully, but these errors were encountered: