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

Various typescript fixes, updated all dependencies, package.json cleanup #49

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

jpoppe
Copy link

@jpoppe jpoppe commented Oct 19, 2018

I had some issues getting this package to compile, got various Typescript errors, etc., probably because I use a recent Node version.

In those commits I have updated all dependencies, fixed the Typescript errors to make it run and added a client command to package.json so you could run the client with ts-node with: yarn run client

src/spec/spec.ts:24:5 - error TS2322: Type 'Promise<Object>' is not assignable to type 'Promise<ApiSpec>'.
  Type 'Object' is not assignable to type 'ApiSpec'.
    The 'Object' type is assignable to very few other types. Did you mean to use the 'any' type instead?
      Property 'host' is missing in type 'Object'
You could now easily test the client with: yarn client
@mikestead
Copy link
Owner

@jpoppe Thanks for these updates. On vacation 🌴 at the moment but will take a look in a week or so when I'm back. On surface all looks good 👍

@mikestead mikestead self-assigned this Oct 20, 2018
@jpoppe
Copy link
Author

jpoppe commented Oct 20, 2018

@jpoppe Thanks for these updates. On vacation at the moment

Don't hurry ;), enjoy your holiday!

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