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

feat: add ECMA Stage 3 decorator support #222

Closed
wants to merge 1 commit into from

Conversation

b3nten
Copy link

@b3nten b3nten commented Jun 27, 2024

This adds an option for enabling ECMA stage 3 decorators and closes #221

@ArnaudBarre
Copy link
Member

ArnaudBarre commented Jun 27, 2024

Thanks for sending a PR, but I don't want to spend too much time maintaining and following variation of the decorators spec. Evan Wallace has done a fantastic job on the subject, and it mostly shows that the feature is not yet standardized across transpilers. I don't want to make to easy for people to depend on some subtile bugs of transpilers, and this why I prefer to not support it for now, so that people can migrate to OXC easily in the future.

@b3nten
Copy link
Author

b3nten commented Jun 27, 2024

Ok that's understandable. Wasn't aware SWC's implementation was buggy. Will use the babel transform until Rolldown drops!

@b3nten b3nten deleted the stage-3-decorators branch June 27, 2024 21:26
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.

Stage 3 decorators throwing syntax error
2 participants