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 build #130

Merged
merged 23 commits into from
Jan 3, 2024
Merged

Update build #130

merged 23 commits into from
Jan 3, 2024

Conversation

MattiasBuelens
Copy link
Owner

  • Remove down-leveling for TypeScript 3.5 or lower. It's no longer worth supporting this.
  • Require Node 18 or higher. The polyfill itself should still work on older Node versions, but we're no longer testing this on the CI.
  • Add polyfill.d.ts for global augmentations to the ReadableStream interface. This allows TypeScript users to use new methods such as ReadableStream.prototype[Symbol.asyncIterator](), even when TypeScript doesn't yet have a built-in type definition for them.
  • Enable verbatimModuleSyntax in the TypeScript configuration. This means we have to explicitly use import type and export type.
  • Manually set Function.name on class methods. This is mostly needed for the ES5 build, the modern builds already have the correct method names.
  • Update a bunch of build dependencies.

@MattiasBuelens MattiasBuelens added the dependencies Pull requests that update a dependency file label Jan 3, 2024
@MattiasBuelens MattiasBuelens merged commit 5bf2685 into master Jan 3, 2024
3 checks passed
@MattiasBuelens MattiasBuelens deleted the update-build branch January 3, 2024 17:18
@MattiasBuelens MattiasBuelens added this to the v3.3.0 milestone Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant