Skip to content

Incompatible ts-loader version #9690

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

Closed
wants to merge 1 commit into from

Conversation

nicholasruunu
Copy link

The ts-loader major version must map to the webpack major version:
TypeStrong/ts-loader#748

Since symfony/encore depends on webpack 3, installing latest will produce the error above.

The ts-loader major version must map to the webpack major version:
TypeStrong/ts-loader#748
@javiereguiluz
Copy link
Member

Nice catch! We merged it on 3.4 branch. Congrats on your first Symfony Docs contribution!

javiereguiluz added a commit that referenced this pull request Apr 27, 2018
This PR was submitted for the 4.0 branch but it was merged into the 3.4 branch instead (closes #9690).

Discussion
----------

Incompatible ts-loader version

The ts-loader major version must map to the webpack major version:
TypeStrong/ts-loader#748

Since symfony/encore depends on webpack 3, installing latest will produce the error above.

Commits
-------

8f2d2f1 Incompatible ts-loader version
@ldawkes
Copy link

ldawkes commented Sep 14, 2018

@javiereguiluz This change was never merged, the docs still reference the un-constrained ts-loader installation.

@javiereguiluz
Copy link
Member

Reopening so I can investigate what happened. Thanks @ldawkes!

@javiereguiluz javiereguiluz reopened this Sep 14, 2018
@javiereguiluz
Copy link
Member

I couldn't fix this error in any way, so I did this change myself again and committed directly to supported branches. The change is now in the repo (see https://github.com/symfony/symfony-docs/blob/master/frontend/encore/typescript.rst) and it will be on symfony.com after the next doc generation. Cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants