-
Notifications
You must be signed in to change notification settings - Fork 49
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
Upgrade Typescript compiler with latest Testcafe version #24
Comments
It's in their newest version (v1.10.2-alpha.1) and I think they will upgrade it soon as in their documents clearly stated. |
tsConfigPath is
So the change you refer to is already available in 1.10.1. As shown in the screenshot, the new option is available, provided that you upgrade your TestCafe dependency. I'm not sure what you are expecting of me here. |
The new compilerOptions have a typescript property which is not present right now in gherkin-testcafe, so we won't be able to pass our configuration file to it. |
Yes! |
That's weird, the only thing I had to do was upgrade testcafe (1.10.1) in my personal project (NOT gherkin-testcafe). Gherkin-testcafe keeps most of the runner object, it just adds a few functions and modifies one. Anything else is exported as is and should work without any change in this package. I'm not sure what I can do, but I'm willing to help you figure this out. |
Sorry I didn't make my point clear enough. In gherkin-testcafe, we have our own compiler which received our configs to pass to testcafe compilers. But in testcafe, they clearly changed how to receive our configs as you can see here. Testcafe get only the typescript property in configuration object to pass down to typescript compiler while gherkin-testcafe get the whole object and pass straight down. |
the options object from the API has to be destructured to retrieve the typescript property and pass it to the typescript compiler, instead of passing the whole object BREAKING CHANGE: the new version will only work with testcafe 1.10 and above #24
Sorry I didn't understand the issue sooner. |
I think the sollution is to change gherkin-testcafe/src/compiler.js Line 58 in c55e0f8
to new TestcafeTypescriptCompiler(compilerOptions.typescript) |
I tested the above change locally and my customized typescript compile config was successfully passed through |
This is totally correct! I have to change it like this locally to pass my config through |
@Arthy000 when will this bug be fixed and released? |
Hello, |
@Arthy000 your branch will fix this issue! |
I just published a new version that contains the fix for this issue, as well as a fix for Unix users |
Recently, TestCafe released a new version (@v1.10.2-alpha.1) and they removed the tsConfigPath function in favour of compilerOptions, also they specify TypeScript option to pass into their TypeScript compiler
It seems that this is the way it is from now on so do you think you can make an upgrade for this? Thank you very much!
The text was updated successfully, but these errors were encountered: