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

fix: creating ast transformer #3

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

Conversation

Farenheith
Copy link
Member

@Farenheith Farenheith commented Jan 19, 2025

Adding ast transformer option for extension methods generation

  • Create transpilation logic
  • Support using extension methods imported from other files
  • Avoid using an extension method if the class already has a method with the same name
  • Figure out a way to help define the extension methods declaration types in a way that the types throw an error when the user tries to create an extension method with the same name as an existing method in the target class
  • Support extension methods to be used from function/methods returns
  • Figure out if there's a way to make extension declarations visible only when the file that declares it is imported

@Farenheith Farenheith marked this pull request as draft January 19, 2025 18:39
@Farenheith Farenheith force-pushed the ast-transformer branch 4 times, most recently from c21d30f to 2600df1 Compare January 19, 2025 18:51
Copy link

@gustavo-rodrigues-dev gustavo-rodrigues-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

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.

2 participants