-
Notifications
You must be signed in to change notification settings - Fork 14
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
default tsconfig in BigTest expects .ts
#959
Comments
We really ought to fix this, and I'm not sure what the options are. Are these some of them?
|
@jbolda do you have an example repo that I can clone with this failing? I have an idea how to fix this. |
@dagda1 I think if you clone frontside.com and delete the bigtest.tsconfig.json you should be able to reproduce. I meant to say that in the OP but apparently missed it. |
I've just tested this PR #961 and it seems to work |
and it was previously displaying the message |
If you init a project without Typescript, it is very likely you won't have / configure a
tsconfig.json
. The default config does not have"allowJs": true
set, so it will fail. To alleviate the issue, I had to make a tsconfig.json including it such as below. It still does appear to run though?You can produce by removing the
bigtest.tsconfig.json
, updatingbigtest.json
to remove that reference, and running the tests. (Note you need toyarn ntl build
andnpx serve public
for the test.)error
created config
The text was updated successfully, but these errors were encountered: