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

Update __spreadArray helper #151

Merged
merged 1 commit into from Jun 11, 2021
Merged

Update __spreadArray helper #151

merged 1 commit into from Jun 11, 2021

Conversation

rbuckton
Copy link
Member

Updates the __spreadArray helper to align with microsoft/TypeScript#44527

@rbuckton
Copy link
Member Author

Can someone review please? We would need to ship an updated tslib before I can merge microsoft/TypeScript#44527, since using a nightly with --importHelpers would result in a checker error until an updated tslib is available.

@rbuckton rbuckton merged commit 1f2daa7 into main Jun 11, 2021
@rbuckton
Copy link
Member Author

@DanielRosenwasser can you let me know when this is published? As far as versioning is concerned, this is a backwards compatible change (the only slight difference would be for [1, , 2, ...[3]], which we would have incorrectly turned into [1, undefined, 2, 3] anways).

@DanielRosenwasser DanielRosenwasser deleted the updateSpreadArray branch June 11, 2021 22:21
@DanielRosenwasser
Copy link
Member

DanielRosenwasser commented Jun 11, 2021

@rbuckton it sounds like for any existing user (ignoring people manually writing out weird references to tslib), this is backwards-compatible. Otherwise, it's just enabling TypeScript 4.4 to make a breaking change. Is that accurate?

@DanielRosenwasser
Copy link
Member

I'm going to version this as 2.3.0.

@rbuckton
Copy link
Member Author

Its not even a breaking change. TS 4.4 would work with old tslib, but would be slower and have the older incorrect array-packing behavior, and TS 4.3 will work with the new tslib and likely be faster but mostly maintaining the older incorrect array-packing behavior.

@DanielRosenwasser
Copy link
Member

2.3.0 should be available shortly.

https://github.com/microsoft/tslib/releases/tag/2.3.0

bors bot added a commit to foosbyte/kicker-trainer that referenced this pull request Oct 19, 2021
1440: fix: update dependency tslib to v2.3.1 (master) r=renovate[bot] a=renovate[bot]

[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [tslib](https://www.typescriptlang.org/) ([source](https://togithub.com/Microsoft/tslib)) | [`2.1.0` -> `2.3.1`](https://renovatebot.com/diffs/npm/tslib/2.1.0/2.3.1) | [![age](https://badges.renovateapi.com/packages/npm/tslib/2.3.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/tslib/2.3.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/tslib/2.3.1/compatibility-slim/2.1.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/tslib/2.3.1/confidence-slim/2.1.0)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>Microsoft/tslib</summary>

### [`v2.3.1`](https://togithub.com/Microsoft/tslib/releases/2.3.1)

[Compare Source](https://togithub.com/Microsoft/tslib/compare/2.3.0...2.3.1)

This release updates the `__spreadArray` helper for TypeScript 4.4 to correctly operate on collections that are not "concat-spreadable" such as the DOM's `NodeList`s and `HTMLCollection`s.

### [`v2.3.0`](https://togithub.com/Microsoft/tslib/releases/2.3.0)

[Compare Source](https://togithub.com/Microsoft/tslib/compare/2.2.0...2.3.0)

This release updates tslib to use TypeScript 4.4's upcoming `__spreadArray` helper which correctly preserves sparse array inputs (e.g. arrays containing "missing" elements like `[1, 2, , 4]`). This new version of `__spreadArray` is backwards-compatible and is often also faster. See [microsoft/tslib#151 for more details.

### [`v2.2.0`](https://togithub.com/Microsoft/tslib/releases/2.2.0)

[Compare Source](https://togithub.com/Microsoft/tslib/compare/2.1.0...2.2.0)

This release supports TypeScript 4.3's new functionality for ECMAScript private methods and accessors, and private static class members.

It does so by expanding the scope of `__classPrivateFieldGet` and `__classPrivateFieldSet`. See [microsoft/tslib#146 for more details.

</details>

---

### Configuration

📅 **Schedule**: At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/foosbyte/kicker-trainer).

Co-authored-by: Renovate Bot <bot@renovateapp.com>
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

2 participants