-
-
Notifications
You must be signed in to change notification settings - Fork 198
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
Support ts-loader 9 #993
Comments
Breaking changes: https://github.com/TypeStrong/ts-loader/blob/main/CHANGELOG.md#v900 What should we do about the Node.js version? Since Encore provides support for Node.js 10. WDYT? |
It can still be supported as with #983, so once that has been merged allowing v9 for ts-loader doesn't seem like a problem. Dropping Node 10 was not the favorite option of @weaverryan due to Ubuntu LTS (see the discussing in #983). |
Hum, it reminds me the discussion on #912... I still don't understand why we would prefer to get stuck with an outdated Node.js version offered by Ubuntu (LTS or not), instead of using the PPA from Node.js, or nvm, or n... 😞 w/e |
It's more: we want people using Ubuntu's LTS to be able to easily use Symfony or Encore, without needing to go outside the normal package manager to install a newer version of PHP or Node. However, I also recognize that Node is especially tricky, since we seem to be an outlier in doing this - most packages drop Node support ASAP :) |
Note that this no longer holds for Symfony 6 as that requires at least PHP 8, which is not available (without PPA) in Ubuntu LTS or the next stable Debian... I know that the choice was done for technical reasons as PHP 8 has several benefits, but dropping an officially unsupported version seems to be a similar choice. |
Like symfony#999, the only breaking change is that the minimum supported version of Node.js is 12.0.0, see https://github.com/TypeStrong/ts-loader/blob/main/CHANGELOG.md#v900
This PR was squashed before being merged into the main branch. Discussion ---------- Allow ts-loader ^9.0.0, close #993 Like #999, the only breaking change is that the minimum supported version of Node.js is 12.0.0, see https://github.com/TypeStrong/ts-loader/blob/main/CHANGELOG.md#v900 **EDIT:** PR #1000 🤯 **EDIT 2:** Looks like something broke. It happens since 9.0.0 (exact version) but I don't know what happens. **EDIT 3:** Added TypeScript types for `.vue`, and it's working fine. Not sure if it's really a BC break, I expect most of people using Vue 3 + TypeScript already using a `vue-shims.d.ts` file... 😕 Commits ------- 4587706 Allow ts-loader ^9.0.0, close #993
ts-loader v9 is currently not supported.
The text was updated successfully, but these errors were encountered: