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

Lit 3.0 branch #3752

Merged
merged 159 commits into from
Oct 9, 2023
Merged

Lit 3.0 branch #3752

merged 159 commits into from
Oct 9, 2023

Conversation

justinfagnani
Copy link
Collaborator

@justinfagnani justinfagnani commented Mar 23, 2023

This is a draft PR that exists just to easily view build status, diffs, and keep GitHub from continuously suggesting that we make a PR for the 3.0 branch.

@changeset-bot
Copy link

changeset-bot bot commented Mar 23, 2023

🦋 Changeset detected

Latest commit: 228ae0f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@lit-labs/compiler Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Mar 23, 2023

📊 Tachometer Benchmark Results

Summary

⏳ Benchmarks are currently running. Results below are out of date.

nop-update

  • lit-html-kitchen-sink: unsure 🔍 -16% - +10% (-3.86ms - +2.47ms)
    this-change vs tip-of-tree

render

  • lit-element-list: 98.01ms - 103.21ms
  • lit-html-kitchen-sink: unsure 🔍 -9% - +6% (-4.43ms - +2.71ms)
    this-change vs tip-of-tree
  • lit-html-repeat: unsure 🔍 -5% - +22% (-0.71ms - +2.99ms)
    this-change vs tip-of-tree
  • lit-html-template-heavy: unsure 🔍 -5% - +1% (-3.31ms - +0.98ms)
    this-change vs tip-of-tree
  • reactive-element-list: unsure 🔍 -4% - +2% (-2.80ms - +1.15ms)
    this-change vs tip-of-tree

update

  • lit-element-list: 1113.71ms - 1132.04ms
  • lit-html-kitchen-sink: unsure 🔍 -10% - +4% (-12.03ms - +4.52ms)
    this-change vs tip-of-tree
  • lit-html-repeat: unsure 🔍 -3% - +3% (-9.31ms - +8.79ms)
    this-change vs tip-of-tree
  • lit-html-template-heavy: unsure 🔍 -2% - +3% (-3.66ms - +4.63ms)
    this-change vs tip-of-tree
  • reactive-element-list: unsure 🔍 -1% - +1% (-15.49ms - +16.02ms)
    this-change vs tip-of-tree

update-reflect

  • lit-element-list: 1084.56ms - 1101.69ms
  • reactive-element-list: unsure 🔍 -2% - +1% (-20.18ms - +10.31ms)
    this-change vs tip-of-tree

Results

⏳ Benchmarks are currently running. Results below are out of date.
lit-element-list

render

VersionAvg timevs
98.01ms - 103.21ms-

update

VersionAvg timevs
1113.71ms - 1132.04ms-

update-reflect

VersionAvg timevs
1084.56ms - 1101.69ms-
lit-html-kitchen-sink

render

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
45.33ms - 50.21ms-unsure 🔍
-9% - +6%
-4.43ms - +2.71ms
unsure 🔍
-6% - +11%
-2.51ms - +4.93ms
tip-of-tree
tip-of-tree
46.02ms - 51.24msunsure 🔍
-6% - +9%
-2.71ms - +4.43ms
-unsure 🔍
-4% - +13%
-1.77ms - +5.90ms
previous-release
previous-release
43.76ms - 49.37msunsure 🔍
-10% - +5%
-4.93ms - +2.51ms
unsure 🔍
-12% - +3%
-5.90ms - +1.77ms
-

update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
107.88ms - 118.76ms-unsure 🔍
-10% - +4%
-12.03ms - +4.52ms
unsure 🔍
-8% - +6%
-9.77ms - +7.28ms
tip-of-tree
tip-of-tree
110.85ms - 123.31msunsure 🔍
-4% - +11%
-4.52ms - +12.03ms
-unsure 🔍
-6% - +10%
-6.54ms - +11.56ms
previous-release
previous-release
108.01ms - 121.13msunsure 🔍
-6% - +9%
-7.28ms - +9.77ms
unsure 🔍
-10% - +6%
-11.56ms - +6.54ms
-

nop-update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
20.51ms - 25.25ms-unsure 🔍
-16% - +10%
-3.86ms - +2.47ms
unsure 🔍
-12% - +16%
-2.56ms - +3.56ms
tip-of-tree
tip-of-tree
21.48ms - 25.67msunsure 🔍
-11% - +17%
-2.47ms - +3.86ms
-unsure 🔍
-8% - +18%
-1.66ms - +4.05ms
previous-release
previous-release
20.44ms - 24.31msunsure 🔍
-15% - +11%
-3.56ms - +2.56ms
unsure 🔍
-17% - +7%
-4.05ms - +1.66ms
-
lit-html-repeat

render

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
13.11ms - 16.67ms-unsure 🔍
-5% - +22%
-0.71ms - +2.99ms
unsure 🔍
-6% - +20%
-0.78ms - +2.84ms
tip-of-tree
tip-of-tree
13.23ms - 14.27msunsure 🔍
-19% - +4%
-2.99ms - +0.71ms
-unsure 🔍
-5% - +4%
-0.74ms - +0.51ms
previous-release
previous-release
13.52ms - 14.20msunsure 🔍
-18% - +4%
-2.84ms - +0.78ms
unsure 🔍
-4% - +5%
-0.51ms - +0.74ms
-

update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
337.52ms - 350.20ms-unsure 🔍
-3% - +3%
-9.31ms - +8.79ms
unsure 🔍
-3% - +2%
-10.02ms - +7.45ms
tip-of-tree
tip-of-tree
337.66ms - 350.58msunsure 🔍
-3% - +3%
-8.79ms - +9.31ms
-unsure 🔍
-3% - +2%
-9.86ms - +7.80ms
previous-release
previous-release
339.13ms - 351.16msunsure 🔍
-2% - +3%
-7.45ms - +10.02ms
unsure 🔍
-2% - +3%
-7.80ms - +9.86ms
-
lit-html-template-heavy

render

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
70.29ms - 73.82ms-unsure 🔍
-5% - +1%
-3.31ms - +0.98ms
unsure 🔍
-2% - +4%
-1.50ms - +2.85ms
tip-of-tree
tip-of-tree
72.00ms - 74.44msunsure 🔍
-1% - +5%
-0.98ms - +3.31ms
-slower ❌
0% - 5%
0.07ms - 3.61ms
previous-release
previous-release
70.10ms - 72.66msunsure 🔍
-4% - +2%
-2.85ms - +1.50ms
faster ✔
0% - 5%
0.07ms - 3.61ms
-

update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
148.68ms - 153.86ms-unsure 🔍
-2% - +3%
-3.66ms - +4.63ms
unsure 🔍
-3% - +2%
-5.32ms - +2.92ms
tip-of-tree
tip-of-tree
147.54ms - 154.02msunsure 🔍
-3% - +2%
-4.63ms - +3.66ms
-unsure 🔍
-4% - +2%
-6.25ms - +2.87ms
previous-release
previous-release
149.27ms - 155.68msunsure 🔍
-2% - +4%
-2.92ms - +5.32ms
unsure 🔍
-2% - +4%
-2.87ms - +6.25ms
-
reactive-element-list

render

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
72.05ms - 74.68ms-unsure 🔍
-4% - +2%
-2.80ms - +1.15ms
unsure 🔍
-7% - +2%
-5.13ms - +1.73ms
tip-of-tree
tip-of-tree
72.71ms - 75.66msunsure 🔍
-2% - +4%
-1.15ms - +2.80ms
-unsure 🔍
-6% - +3%
-4.37ms - +2.62ms
previous-release
previous-release
71.90ms - 78.23msunsure 🔍
-2% - +7%
-1.73ms - +5.13ms
unsure 🔍
-4% - +6%
-2.62ms - +4.37ms
-

update

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
1155.89ms - 1177.26ms-unsure 🔍
-1% - +1%
-15.49ms - +16.02ms
unsure 🔍
-2% - +0%
-29.17ms - +5.05ms
tip-of-tree
tip-of-tree
1154.74ms - 1177.89msunsure 🔍
-1% - +1%
-16.02ms - +15.49ms
-unsure 🔍
-3% - +0%
-30.00ms - +5.36ms
previous-release
previous-release
1165.27ms - 1192.00msunsure 🔍
-0% - +3%
-5.05ms - +29.17ms
unsure 🔍
-0% - +3%
-5.36ms - +30.00ms
-

update-reflect

VersionAvg timevs this-change
vs tip-of-tree
tip-of-tree
vs previous-release
previous-release
this-change
1160.76ms - 1183.83ms-unsure 🔍
-2% - +1%
-20.18ms - +10.31ms
unsure 🔍
-2% - +1%
-19.00ms - +11.39ms
tip-of-tree
tip-of-tree
1167.26ms - 1187.20msunsure 🔍
-1% - +2%
-10.31ms - +20.18ms
-unsure 🔍
-1% - +1%
-12.91ms - +15.17ms
previous-release
previous-release
1166.21ms - 1185.98msunsure 🔍
-1% - +2%
-11.39ms - +19.00ms
unsure 🔍
-1% - +1%
-15.17ms - +12.91ms
-

tachometer-reporter-action v2 for Benchmarks

justinfagnani and others added 26 commits March 24, 2023 12:23
* Remove sauce-ie11 workflow
* Remove sauce-ie11 browser preset
* Remove ie11 and edge launcher examples in starter kits
* Remove decorators export from lit-element/index.ts
* Remove UpdatingElement alias for ReactiveElement
* Remove experimental-hydrate-support from lit-element

* Remove expriemental-hydrate from lit-html

* Remove re-exports from lit

* Remove from node-imports tests
* Coerce toggleAttribute's second arg to boolean

Otherwise binding the value `undefined` to a boolean part would mean `toggle` rather than `false`

* Add a test for a boolean part with undefined
* Use Symbol.for in dev mode

This makes it much easier to maintain state for an element during HMR because keys stay the same across different versions of the class.
* First part of conversion to TS 5.0

* Minor fixes

* Changeset

* Remove unnecessary dep

* Fix another compiler error.

* Do a breaking change bump to localize-tools and ts-transformers.

* Standardize on a single version for @esm-bundle/chai

* Uninstall chai
Note: The size bump here is correcting the 3.0 branch. Because merging into 3.0 is not atomic, it's possible that two changes both updated sizes, but no change existed to combine the two changes.
…nalized (#4257)

* elementProperties is a static property that's dynamic similarly to finalized

To get correct semantics out of it, we need to hide it from closure's instinct to collapse it by using dynamic property access syntax.

* Avoid using dynamic property access

Also hide the static property initialization from jscompiler, and it won't manually copy those fields down onto subclasses.

This is a bit larger for terser, but we expect the code to be faster because we're doing fewer dynamic property accesses, and it's both smaller and faster for jscompiler because it can rename both elementProperties and finalize.
* update goldens

* add empty changeset
* Add a missing dependency

gen-manifest depends on being able to resolve LitElement types, so it lit-element must build before it does

* react:test:dev reads index.js so it depends on build:rollup

* Update file reference in @lit-internal/gen-wrapper-vue-test-output

* Fix wrapper-react, use more explicit semver ranges

* Add another missing dep.

* Remove console log

* Add missing dev dep, improve wireit dep.

* Bump eleventy test timeout

This test frequently times out

* Run local tests serially to reduce flakiness.
@rictic rictic changed the title Lit 3.0 Placeholder Lit 3.0! Oct 7, 2023
@rictic rictic marked this pull request as ready for review October 7, 2023 01:18
Copy link
Contributor

@AndrewJakubowicz AndrewJakubowicz left a comment

Choose a reason for hiding this comment

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

Woooooooo! Nice job y'all!

@rictic rictic changed the title Lit 3.0! Lit 3.0 branch Oct 9, 2023
@rictic rictic merged commit 9f9120f into main Oct 9, 2023
9 of 10 checks passed
@rictic rictic deleted the 3.0 branch October 9, 2023 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants