Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Babel 8] Throw decorators and TS plugins in wrong order#17542

Open
liuxingbaoyu wants to merge 2 commits intobabel:mainfrom
liuxingbaoyu:ts-dec-wrong-plugin-order
Open

[Babel 8] Throw decorators and TS plugins in wrong order#17542
liuxingbaoyu wants to merge 2 commits intobabel:mainfrom
liuxingbaoyu:ts-dec-wrong-plugin-order

Conversation

@liuxingbaoyu
Copy link
Member

Q                       A
Fixed Issues? Fixes #17541
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass?
Documentation PR Link
Any Dependency Changes?
License MIT

@liuxingbaoyu liuxingbaoyu added the PR: Bug Fix 🐛 A type of pull request used for our changelog categories label Oct 5, 2025
@liuxingbaoyu
Copy link
Member Author

This may cause breaking changes for some users, but I think it's better than having potential issues.

@babel-bot
Copy link
Collaborator

babel-bot commented Oct 5, 2025

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/60322

@JLHwung
Copy link
Contributor

JLHwung commented Oct 12, 2025

Could you update the failing decorators 2018-12 test case?

@liuxingbaoyu
Copy link
Member Author

preactjs/preset-vite#170 (comment)
This probably can't be merged and I'm afraid of breaking a running project.

@liuxingbaoyu liuxingbaoyu force-pushed the ts-dec-wrong-plugin-order branch from 99337ff to 4baa1bf Compare November 28, 2025 20:30
@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 29, 2025

Open in StackBlitz

commit: 0a78c2d

@liuxingbaoyu liuxingbaoyu force-pushed the ts-dec-wrong-plugin-order branch from 90797f1 to af85087 Compare November 30, 2025 09:19
@liuxingbaoyu liuxingbaoyu changed the title fix: Throwing decorators and TS plugins in wrong order [Babel 8] Throwing decorators and TS plugins in wrong order Nov 30, 2025
@liuxingbaoyu liuxingbaoyu changed the title [Babel 8] Throwing decorators and TS plugins in wrong order [Babel 8] Throw decorators and TS plugins in wrong order Nov 30, 2025
@u5193919988-debug
Copy link

thank you nice work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚠️ Do not merge PR: Bug Fix 🐛 A type of pull request used for our changelog categories Spec: Decorators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proposal decorators initializers not called if class contains an abstract field

4 participants