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

[bug] SplitTooLongLine removes all but first returned value #68

Closed
adrszad opened this issue May 19, 2021 · 3 comments · Fixed by #72
Closed

[bug] SplitTooLongLine removes all but first returned value #68

adrszad opened this issue May 19, 2021 · 3 comments · Fixed by #72
Assignees
Labels
bug Something isn't working
Milestone

Comments

@adrszad
Copy link
Contributor

adrszad commented May 19, 2021

Consider line:
${var1} ${var2} Some Lengthy Keyword So That This Line Is To Long ${arg1} ${arg2}

it's transformed to:
${var1} Some Lengthy Keyword So That This Line Is To Long
... ${arg1} ${arg2}

@adrszad
Copy link
Contributor Author

adrszad commented May 27, 2021

you can also consider a case when line should be split before a keyword because of multiple assignments or long variable names on assignment:
${returned_value_with_long_name_1} ${returned_value_with_long_name_2} ${returned_value_with_long_name_3} ${returned_value_with_long_name_4} ${returned_value_with_long_name_5} ${returned_value_with_long_name_6} some keyword

@bhirsz
Copy link
Member

bhirsz commented May 29, 2021

Thanks for the report. I've tested it and I can also confirm this bug. I will take a look into fixing it. While working at it I will also consider problems with long variable names.

@bhirsz
Copy link
Member

bhirsz commented May 29, 2021

Fixed in #72. I've created separate issue for splitting on too long assignment variable names.

@bhirsz bhirsz added this to the 1.2.0 milestone Jun 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants