Skip to content

fix: move transform-class-properties to presets#97

Merged
ValeraS merged 1 commit intomainfrom
fix-move-plugin-transform-class-properties-to-presets
Nov 1, 2023
Merged

fix: move transform-class-properties to presets#97
ValeraS merged 1 commit intomainfrom
fix-move-plugin-transform-class-properties-to-presets

Conversation

@melikhov-dev
Copy link
Copy Markdown
Contributor

@melikhov-dev melikhov-dev commented Nov 1, 2023

Because we need to run transform after @babel/preset-typescript

SyntaxError: TypeScript 'declare' fields must first be transformed by @babel/plugin-transform-typescript.
If you have already enabled that plugin (or '@babel/preset-typescript'), make sure that it runs before any plugin related to additional class features:

  • @babel/plugin-transform-class-properties
  • @babel/plugin-transform-private-methods
  • @babel/plugin-proposal-decorators

Plugin ordering:

Plugins run before Presets.
Plugin ordering is first to last.
Preset ordering is reversed (last to first).

https://babeljs.io/docs/plugins#plugin-ordering

@melikhov-dev melikhov-dev force-pushed the fix-move-plugin-transform-class-properties-to-presets branch from 19a1590 to 7e4cb1b Compare November 1, 2023 13:12
@melikhov-dev melikhov-dev requested a review from ValeraS November 1, 2023 13:19
@ValeraS ValeraS merged commit 1a6b7ea into main Nov 1, 2023
@ValeraS ValeraS deleted the fix-move-plugin-transform-class-properties-to-presets branch November 1, 2023 14:37
@gravity-ui gravity-ui Bot mentioned this pull request May 6, 2026
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