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

Typescript definition doesn't seem to be correct. #11

Closed
dperetti opened this issue Dec 22, 2019 · 11 comments
Closed

Typescript definition doesn't seem to be correct. #11

dperetti opened this issue Dec 22, 2019 · 11 comments

Comments

@dperetti
Copy link
Contributor

import odiff from 'odiff'

odiff.equal()

-> Property 'equal' does not exist on type '(a: any, b: any) => odiffResult[]'

@fresheneesz
Copy link
Contributor

Ah hm, would you mind testing if the following works?

import * as odiff from 'odiff'

@dperetti
Copy link
Contributor Author

Doesn't work either.

@fresheneesz
Copy link
Contributor

Sorry, I don't use typescript. @tontonrally added the code for that, so maybe he can help? Otherwise you might need to do your own investigating. I'm happy to help. I'd recommend seeing what is inside odiff in the two cases you tried up there (if 'equal' doesn't exist, is it null, is it an emtpy object, something else?)

@dperetti
Copy link
Contributor Author

👉 #12

@fresheneesz
Copy link
Contributor

👉😎👉

@fresheneesz
Copy link
Contributor

Published v1.4.0 with this in it

@tontonrally
Copy link

Sorry guys, didn't see that... As it is now fixed, I don't need to dig into. Thanks @dperetti for the fix.

@chapmanjacobd
Copy link
Contributor

type 'rm' has vals: any[]

@fresheneesz
Copy link
Contributor

@chapmanjacobd I think you're right. Would you be willing to write a pull request for that?

@chapmanjacobd
Copy link
Contributor

chapmanjacobd commented Jun 5, 2020 via email

@fresheneesz
Copy link
Contributor

Fixed in pull request #19

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

No branches or pull requests

4 participants