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

Add test workspaces and examples to monorepo design #149

Merged
merged 7 commits into from
Mar 6, 2023

Conversation

lugehorsam
Copy link
Contributor

@lugehorsam lugehorsam commented Mar 3, 2023

  • Add test workspaces and examples to monorepo design
  • Adds skipLibCheck to the tsconfig. By default, tsc tries to compile type definitions for every single package in the workspace, regardless of which package you are building. This meant it was trying to compile Pupeteer (nodejs-based) for nakama-js, which failed compilation because of node-specific globals. skipLibCheck sounds bad but all it means is "only check type definitions that are actually used by the project being compiled." Why this isn't a default, I don't know.
  • Removes window. prefix for setTimeout calls for portability with NodeJS and Dino.

There are 5 remaining tests of 140~ that do not pass. I am tracking them in a separate issue.

@lugehorsam lugehorsam requested a review from tomglenn March 3, 2023 18:16
@lugehorsam lugehorsam merged commit 5382b9d into master Mar 6, 2023
@lugehorsam lugehorsam deleted the luke/test-workspaces branch March 6, 2023 14:12
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