Skip to content

Conversation

Timer
Copy link
Contributor

@Timer Timer commented Oct 24, 2018

This causes weird behavior that allows the user to think types are being checked when they're really not.

@Timer Timer added this to the 2.1 milestone Oct 24, 2018
@Timer Timer mentioned this pull request Oct 24, 2018
3 tasks
@gaearon
Copy link
Contributor

gaearon commented Oct 24, 2018

👋 I'm that user

@gaearon
Copy link
Contributor

gaearon commented Oct 24, 2018

Can we also show some error?

@Timer
Copy link
Contributor Author

Timer commented Oct 24, 2018

How about this?

@Timer
Copy link
Contributor Author

Timer commented Oct 24, 2018

Actually, that's goofy. We can do better.

@Timer
Copy link
Contributor Author

Timer commented Oct 24, 2018

How about this?

Second run only shows this:

@Timer Timer changed the title Don't resolve TS files if it's not detected in the project Automatically setup TypeScript when detected Oct 24, 2018
@Timer Timer merged commit 5ead915 into facebook:master Oct 24, 2018
zmitry pushed a commit to zmitry/create-react-app that referenced this pull request Oct 29, 2018
* Don't resolve TS files if it's not detected in the project * Automatically create tsconfig.json file for user * Remove always false check * Add missing file * Don't filter paths too early
nate770 pushed a commit to ONTW/create-react-app that referenced this pull request Oct 30, 2018
* Don't resolve TS files if it's not detected in the project * Automatically create tsconfig.json file for user * Remove always false check * Add missing file * Don't filter paths too early
@lock lock bot locked and limited conversation to collaborators Jan 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.