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

Insert newly generated members at end #692

Merged

Conversation

datho7561
Copy link
Contributor

For the "generate missing member" qute-ls CodeAction, generate the missing member at the end of the respective list. i.e. place the new field at the end of the list of fields, or place the new getter method at the end of the list of methods.

Signed-off-by: David Thompson davthomp@redhat.com

@datho7561 datho7561 force-pushed the generate-member-at-end branch 8 times, most recently from f7fa6f0 to 3f78ed6 Compare January 16, 2023 19:00
@datho7561 datho7561 marked this pull request as ready for review January 16, 2023 19:54
@datho7561
Copy link
Contributor Author

In addition the the feature improvements, I tried to clean up the code a bit.

For the "generate missing member" qute-ls CodeAction,
generate the missing member at the end of the respective list.
i.e. place the new field at the end of the list of fields,
or place the new getter method at the end of the list of methods.

Signed-off-by: David Thompson <davthomp@redhat.com>
@angelozerr
Copy link
Contributor

LGTM. Thanks @datho7561 !

@angelozerr angelozerr merged commit ff56ef1 into redhat-developer:master Jan 30, 2023
@angelozerr angelozerr added this to the 0.14.0 milestone Jan 30, 2023
@datho7561 datho7561 deleted the generate-member-at-end branch January 30, 2023 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants