Skip to content

Es private methods and accessors simplify emit #56

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

Conversation

dragomirtitian
Copy link

No description provided.

… and modified the existing helpers for get and set fields to do the same job.
@dragomirtitian dragomirtitian force-pushed the es-private-methods-and-accessors-simplify-emit branch 2 times, most recently from e605e6f to 6c19c52 Compare March 18, 2021 08:03
@dragomirtitian dragomirtitian force-pushed the es-private-methods-and-accessors-simplify-emit branch 2 times, most recently from 46573de to eed5e6b Compare March 18, 2021 13:03
Copy link

@mkubilayk mkubilayk left a comment

Choose a reason for hiding this comment

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

🎉 Transform code looks good. I left some comments but they are all cosmetic changes and mostly optional. Will take a look at fixtures now.

@dragomirtitian dragomirtitian force-pushed the es-private-methods-and-accessors-simplify-emit branch from eed5e6b to a3fdc4b Compare March 18, 2021 14:21
@dragomirtitian dragomirtitian merged commit 668245b into es-private-methods-and-accessors Mar 18, 2021
@dragomirtitian dragomirtitian deleted the es-private-methods-and-accessors-simplify-emit branch November 17, 2023 15:56
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