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 TypeScript config in typechecking mode #8

Merged
merged 2 commits into from
Mar 16, 2020

Conversation

nwalters512
Copy link
Contributor

As described in microsoft/TypeScript#36917, recent versions of TypeScript don't allow the --noEmit flag to be set alongside "composite": true. This PR updates the typecheck script to pass --composite false to tsc to fix this.

Side note: what do you think about committing a package-lock.json file to the repo? That would help ensure that everyone is working with the expected versions of dev dependencies.

@vercel
Copy link

vercel bot commented Mar 16, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/zdx/useisinviewport/h94cgmc2y
✅ Preview: https://useisinviewport-git-fork-nwalters512-fix-typescript-config.zdx.now.sh

Copy link
Owner

@zeusdeux zeusdeux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd recommend removing the composite key altogether. I don't recall why I put it in there to begin with 😅

@zeusdeux
Copy link
Owner

I think you might need to push again. CircleCI seems to have died :\

@zeusdeux
Copy link
Owner

zeusdeux commented Mar 16, 2020

Oh and as for the package-lock.json, sure! I'll add it in after we merge this.

@zeusdeux
Copy link
Owner

Thanks for this @nwalters512!

@nwalters512 nwalters512 deleted the fix/typescript-config branch March 16, 2020 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants