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

fix: Remove default esModuleInterop option from ts-node and preprocessor #8143

Merged
merged 47 commits into from
Aug 10, 2020

Conversation

chrisbreiding
Copy link
Contributor

@chrisbreiding chrisbreiding commented Jul 31, 2020

User facing changelog

  • BREAKING CHANGE: For TypeScript compilation of spec/support files and the plugins file, the esModuleInterop option is no longer coerced to true. If you need to utilize esModuleInterop, set it in your tsconfig.json.

Additional details

How has the user experience changed?

PR Tasks

@flotwig flotwig mentioned this pull request Aug 4, 2020
21 tasks
@jennifer-shehane
Copy link
Member

@chrisbreiding Nice. I would love links to any docs PRs that are needed or notes you would add for the Migration Guide. The below was previously mentioned for the Migration guide

When a user doesn't have a tsconfig.json file or define esModuleInterop: true, Cypress didn't fail in 4.x, but it might fail after 5.0. Because Cypress now uses the default esModuleInterop setting of TypeScript.

I used 'might', because it wouldn't fail when a user didn't use import syntax.
#7808 (comment)

But providing an example of before / after would be great.

@chrisbreiding chrisbreiding marked this pull request as ready for review August 4, 2020 14:24
@chrisbreiding
Copy link
Contributor Author

@jennifer-shehane Submitted documentation PR: cypress-io/cypress-documentation#3053

@brian-mann brian-mann changed the base branch from tr-49-default-preprocessor-webpack to v5.0-release August 10, 2020 20:58
…esmodule-interop-default

# Conflicts:
#	packages/server/__snapshots__/1_typescript_spec_support_spec.ts.js
#	packages/server/package.json
#	yarn.lock
@brian-mann brian-mann merged commit daeab10 into v5.0-release Aug 10, 2020
@chrisbreiding chrisbreiding deleted the tr-132-change-esmodule-interop-default branch April 5, 2022 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: breaking change Requires a new major release version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants