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

3517-onstruct #3518

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

3517-onstruct #3518

wants to merge 8 commits into from

Conversation

glelouet
Copy link

@glelouet glelouet commented Oct 7, 2023

feature description : #3517

glelouet added 5 commits October 6, 2023 19:41
modified annotation to have shorter name for prefix, suffix, added option to camelcase the variable name and the method name. Created util to extract the var name and met name from the requested name and the annotation.
Changed the tests to have the var match the getter, and implemented the var name  and assignment replacement in the node.
glelouet and others added 3 commits October 15, 2023 14:25
removed asserts as they are not used.
+minor fixes
added source type go deduce method name.
method options can still be overwritten.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant