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

add support for pure annotations class expressions #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

fubhy
Copy link

@fubhy fubhy commented Jul 20, 2024

This adds support for a common pattern we use in @effect/schema.

I couldn't get the babel7 tests to pass because the output I get there is identical to the one for babel6 (without the line breaks). Can you guide me to the right babel version that is needed to reproduce that? If necessary? :-)

Comment on lines +7 to +9
class MyClass extends
/*#__PURE__*/
base() {}
Copy link
Author

@fubhy fubhy Jul 20, 2024

Choose a reason for hiding this comment

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

I just guessed that this would be the output expected by the test suite. But I can't reproduce this formatting with a fresh install of the dependencies here (due to no lock file) :-)

Copy link
Owner

Choose a reason for hiding this comment

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

Well, this repo could use some work on setting up github actions instead of Travis, updating deps and just dropping Babel 6 support 😅

src/index.js Outdated Show resolved Hide resolved
Co-authored-by: Mateusz Burzyński <mateuszburzynski@gmail.com>
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