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

[WIP] Unify JIT and AOT code paths #4188

Merged
merged 13 commits into from
Apr 30, 2021
Merged

[WIP] Unify JIT and AOT code paths #4188

merged 13 commits into from
Apr 30, 2021

Conversation

RobinMalfait
Copy link
Member

@RobinMalfait RobinMalfait commented Apr 26, 2021

WIP start by unifying JIT and AOT codepaths.

@adamwathan adamwathan changed the title cleanup codepaths [WIP] Unify JIT and AOT code paths Apr 26, 2021
@codecov-commenter
Copy link

codecov-commenter commented Apr 28, 2021

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 66.47399% with 348 lines in your changes missing coverage. Please review.

Project coverage is 70.08%. Comparing base (3843700) to head (33df925).

Files with missing lines Patch % Lines
src/plugins/index.js 33.82% 45 Missing ⚠️
src/util/pluginUtils.js 40.98% 32 Missing and 4 partials ⚠️
src/plugins/inset.js 61.42% 23 Missing and 4 partials ⚠️
src/plugins/gradientColorStops.js 63.46% 17 Missing and 2 partials ⚠️
src/plugins/divideWidth.js 59.52% 13 Missing and 4 partials ⚠️
src/plugins/divideColor.js 60.97% 11 Missing and 5 partials ⚠️
src/plugins/fontSize.js 63.63% 11 Missing and 5 partials ⚠️
src/plugins/space.js 62.50% 11 Missing and 4 partials ⚠️
src/util/createUtilityPlugin.js 66.66% 10 Missing and 4 partials ⚠️
src/plugins/boxShadow.js 59.37% 11 Missing and 2 partials ⚠️
... and 22 more

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #4188       +/-   ##
===========================================
- Coverage   86.84%   70.08%   -16.77%     
===========================================
  Files         341      219      -122     
  Lines        4836     5456      +620     
  Branches      919     1032      +113     
===========================================
- Hits         4200     3824      -376     
- Misses        561     1469      +908     
- Partials       75      163       +88     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@adamwathan adamwathan merged commit bfc6162 into master Apr 30, 2021
@adamwathan adamwathan deleted the cleanup-codepaths branch April 30, 2021 14:58
adamwathan added a commit that referenced this pull request May 7, 2021
* WIP

* WIP

* Finish combining JIT and AOT plugins

Still lots of clean up that can be done in some of the more complex ones, but at least it's one file per plugin now.

* Remove unused import

* Fix AOT generation bugs

* Move corePlugins/index.js to corePlugins.js

* Convert JIT files to ESM

* Move tests

* Reorder core plugins to match JIT order

* Update AOT apply tests

* Unify utils

* Combine plugin lists to one single source of truth

* Finish resolving merge conflicts, fix tests

Co-authored-by: Adam Wathan <[email protected]>
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.

3 participants