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

Merged
merged 4 commits into from
Jan 7, 2025

Conversation

fubhy
Copy link
Contributor

@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
Contributor 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
@Andarist Andarist merged commit dc0d53e into Andarist:main Jan 7, 2025
1 check passed
@fubhy fubhy deleted the support-class-expressions branch January 7, 2025 09:50
@fubhy
Copy link
Contributor Author

fubhy commented Jan 7, 2025

Thanks Mateusz! :-)

@Andarist
Copy link
Owner

Andarist commented Jan 7, 2025

Better late than never, right? :P

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