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

Use imported Readable flow type instead of global #6910

Merged
merged 2 commits into from Sep 11, 2021
Merged

Conversation

mischnic
Copy link
Member

The last successful nightly release was 10 days ago, because the automatically generated TS definitions used stream$Readable (which isn't valid) and therefore the check-ts scripts failed.

Instead use the Readable import as we do everywhere else

@height
Copy link

height bot commented Sep 11, 2021

Link Height tasks by mentioning a task ID in the pull request title or description, commit messages, or comments.

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@parcel-benchmark
Copy link

parcel-benchmark commented Sep 11, 2021

Benchmark Results

Kitchen Sink ✅

Timings

Description Time Difference
Cold 1.64s +4.00ms
Cached 262.00ms -13.00ms

Cold Bundles

No bundle changes detected.

Cached Bundles

Bundle Size Difference Time Difference
dist/legacy/parcel.7cdb0fad.webp 102.94kb +0.00b 689.00ms +161.00ms ⚠️
dist/legacy/parcel.7cdb0fad.webp 102.94kb +0.00b 690.00ms +161.00ms ⚠️
dist/modern/parcel.7cdb0fad.webp 102.94kb +0.00b 690.00ms +161.00ms ⚠️

React HackerNews ✅

Timings

Description Time Difference
Cold 8.46s -19.00ms
Cached 415.00ms -3.00ms

Cold Bundles

No bundle changes detected.

Cached Bundles

No bundle changes detected.

AtlasKit Editor ✅

Timings

Description Time Difference
Cold 56.51s -349.00ms
Cached 1.52s -4.00ms

Cold Bundles

Bundle Size Difference Time Difference
dist/index.167ca00c.js 1.78mb -6.00b 🚀 16.10s -146.00ms
dist/index.b4b95a88.js 694.72kb -30.00b 🚀 43.43s +47.00ms
dist/EmojiPickerComponent.34b6043b.js 146.67kb +0.00b 17.48s -9.07s 🚀
dist/esm.723d2911.js 33.15kb +0.00b 17.48s -9.07s 🚀
dist/DatePicker.c8916da2.js 22.98kb +0.00b 17.48s -9.07s 🚀
dist/js.0f0bb621.js 17.25kb +0.00b 31.89s -11.04s 🚀
dist/png-chunks-extract.94b5b9fc.js 3.58kb +0.00b 31.89s -11.04s 🚀
dist/feedback.e18b45f0.js 1.77kb +0.00b 17.48s -9.07s 🚀
dist/workerHasher.567cfc57.js 1.63kb +0.00b 17.48s -9.07s 🚀
dist/heading6.b3b946d1.js 1.51kb +0.00b 17.48s -9.07s 🚀
dist/heading5.e617db66.js 1.38kb +0.00b 17.47s -9.07s 🚀
dist/expand.1f17de7f.js 1.29kb +0.00b 17.48s -9.07s 🚀

Cached Bundles

Bundle Size Difference Time Difference
dist/index.4d511fc9.js 1.78mb -4.00b 🚀 16.06s -472.00ms
dist/index.781cdfc3.js 694.71kb -4.00b 🚀 43.43s -713.00ms

Three.js ✅

Timings

Description Time Difference
Cold 6.04s +24.00ms
Cached 355.00ms +15.00ms

Cold Bundles

No bundle changes detected.

Cached Bundles

No bundle changes detected.

Click here to view a detailed benchmark overview.

@devongovett devongovett merged commit a911372 into v2 Sep 11, 2021
@devongovett devongovett deleted the fix-release branch September 11, 2021 16:54
lettertwo added a commit that referenced this pull request Sep 13, 2021
* v2: (42 commits)
  Remove dead link to docs in diagnostic (#6913)
  Fix engines.parcel in SVG packager (#6911)
  Use imported Readable flow type instead of global (#6910)
  Use non-deprecated SVGO options in HTMLNanoOptimizer (#6785)
  Use yarnpkg registry for all the things (#6908)
  Fix issue about loading configs from tsconfig.json (#6881)
  Add `@section` to Compressor type (#6885)
  Parcel API improvements (#6866)
  Use stream-browserify for polyfilling instead (#6863)
  Add support for compressor plugins (#6776)
  Support for inline style attributes and inline scripts in SVG (#6797)
  Bump less (#6852)
  Make sure (non-React) SVGs are in separate bundles (#6757)
  Add support for injecting manifest into service workers (#6798)
  Fix 'does not export default' error with scope hoisting and url/worklet pipeline (#6803)
  Remaining cargo clippy fixes (#6829)
  Lazily install sharp only when needed (#6816)
  More cargo clippy suggestions (#6811)
  Don't emit runtime manifest for inline child bundles (#6807)
  Apply suggestions from rust-clippy (#6256)
  ...
lettertwo added a commit that referenced this pull request Sep 13, 2021
* v2: (42 commits)
  Remove dead link to docs in diagnostic (#6913)
  Fix engines.parcel in SVG packager (#6911)
  Use imported Readable flow type instead of global (#6910)
  Use non-deprecated SVGO options in HTMLNanoOptimizer (#6785)
  Use yarnpkg registry for all the things (#6908)
  Fix issue about loading configs from tsconfig.json (#6881)
  Add `@section` to Compressor type (#6885)
  Parcel API improvements (#6866)
  Use stream-browserify for polyfilling instead (#6863)
  Add support for compressor plugins (#6776)
  Support for inline style attributes and inline scripts in SVG (#6797)
  Bump less (#6852)
  Make sure (non-React) SVGs are in separate bundles (#6757)
  Add support for injecting manifest into service workers (#6798)
  Fix 'does not export default' error with scope hoisting and url/worklet pipeline (#6803)
  Remaining cargo clippy fixes (#6829)
  Lazily install sharp only when needed (#6816)
  More cargo clippy suggestions (#6811)
  Don't emit runtime manifest for inline child bundles (#6807)
  Apply suggestions from rust-clippy (#6256)
  ...
lettertwo added a commit that referenced this pull request Sep 13, 2021
* bdo/number-edgetypes: (74 commits)
  Remove dead link to docs in diagnostic (#6913)
  Fix engines.parcel in SVG packager (#6911)
  Use imported Readable flow type instead of global (#6910)
  Use non-deprecated SVGO options in HTMLNanoOptimizer (#6785)
  Use yarnpkg registry for all the things (#6908)
  Fix issue about loading configs from tsconfig.json (#6881)
  Add `@section` to Compressor type (#6885)
  Parcel API improvements (#6866)
  Use stream-browserify for polyfilling instead (#6863)
  Add support for compressor plugins (#6776)
  Support for inline style attributes and inline scripts in SVG (#6797)
  Bump less (#6852)
  Make sure (non-React) SVGs are in separate bundles (#6757)
  Add support for injecting manifest into service workers (#6798)
  Fix 'does not export default' error with scope hoisting and url/worklet pipeline (#6803)
  Remaining cargo clippy fixes (#6829)
  Lazily install sharp only when needed (#6816)
  More cargo clippy suggestions (#6811)
  Don't emit runtime manifest for inline child bundles (#6807)
  Apply suggestions from rust-clippy (#6256)
  ...
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.

None yet

3 participants