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

Downstreamed changes from @embroider/[email protected] #217

Merged
merged 3 commits into from
Dec 20, 2023

Conversation

ijlee2
Copy link
Owner

@ijlee2 ijlee2 commented Dec 20, 2023

Description

By replacing a few Babel plugins with decorator-transforms, we can ship fewer bytes and avoid the need to transpile other class features.

Before After
Package size 15.7 kB 14.6 kB
Unpacked size 54.3 kB 49.6 kB
Total files 40 38

@ijlee2 ijlee2 added the enhance: dependency Issue asks for a new or updated dependency label Dec 20, 2023
Comment on lines +19 to +26
[
"module:decorator-transforms",
{
"runtime": {
"import": "decorator-transforms/runtime"
}
}
]
Copy link
Owner Author

Choose a reason for hiding this comment

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

@@ -4,6 +4,7 @@
"allowImportingTsExtensions": true,
"allowJs": true,
"declarationDir": "declarations",
"rootDir": "./src",
Copy link
Owner Author

Choose a reason for hiding this comment

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

@@ -2,6 +2,7 @@
// These will *not* be published as part of your addon, so be careful that your published code does not rely on them!

import '@glint/environment-ember-loose';
import '@glint/environment-ember-template-imports';
Copy link
Owner Author

Choose a reason for hiding this comment

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

@ijlee2 ijlee2 merged commit b4f40d4 into main Dec 20, 2023
28 checks passed
@ijlee2 ijlee2 deleted the downstream-changes-from-addon-blueprint branch December 20, 2023 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhance: dependency Issue asks for a new or updated dependency
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant