-
Notifications
You must be signed in to change notification settings - Fork 4
Update Node.js version #36
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
Conversation
…release workflow for clarity
…target to esnext in tsconfig.json
Sidnioulz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, we can then close #35 once this is merged
…command execution
…e tsup.config.ts in the TypeScript compilation
|
@Sidnioulz I'm at a loss at that error in release... What's going on ? Have you seen this before? |
…json and pnpm-lock.yaml for compatibility reasons
…wJs and moduleDetection options, and clean up unused settings for improved TypeScript configuration.
…on and pnpm-lock.yaml to ensure compatibility with the latest features and improvements.
| "moduleDetection": "force", | ||
| "moduleResolution": "bundler", | ||
| "module": "preserve", | ||
| "module": "es2015", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ndelangen I'm wondering if es2015 is actually supported or not. I think we could've kept preserve here, that's what we have in the addon-kit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It isn't used by esbuild anyway AFAIK
📦 Published PR as canary version:
2.0.1--canary.36.f3dd7e7.0✨ Test out this PR locally via: