-
Notifications
You must be signed in to change notification settings - Fork 17
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
[CSR-635] Support ESM #125
Comments
A must have, using
|
please check out https://github.com/currents-dev/cypress-cloud/releases/tag/v1.8.0-beta.5
|
@agoldis unfortunately for me, it's still reproducing. |
@MadSandwich |
@agoldis unfortunately I can't share such logs, they have information that I couldn't share. But I've created a minimal reproducible example. And here the error that I've receive with such setup, I have same configurations on my real repo and same errors, so that probably could help to find where an actual problem. (Maybe I missing something. All settings in the config was set to likely non-existing, just to show an error) |
@MadSandwich please remove the sensitive information, it'd greatly speed up the troubleshooting. And thanks for sharing the exampl |
Here is the simple video of trying 2 ways of configuration. @agoldis, I've made an repo where u can check everything you need, because, I not sure, that I cover all needed for you details. Hope it will help u more. |
@MadSandwich can you please enable the debug mode and share the logs from the repo you created. Thank you |
|
@MadSandwich Please try using [email protected], should be good for ESM on Windows |
Everything looks good. Thanks! The issue can be closed after merging this fix. |
Description
Support ESM to parse path correctly.
(I hope I describe it at least understandable =) )
Example usage
Set
type:module
in package.json file and use default config to run tests:The text was updated successfully, but these errors were encountered: