Not compatible with @types/relay-runtime 1.3.9 about relay-compiler-language-typescript HOT 10 CLOSED

DT typings have no notion of semver.

I currently do not have any spare time to spend on this, so I'd love somebody else to PR a solution. I agree with @mattkrick that it is likely we'll need separate types for <2 and >=2, but that might be a bit much to tackle now, so for expediency sake perhaps most of those changes should be reverted for now?

from relay-compiler-language-typescript.

@steida Solution coming as relaxed typings v1.3.12
https://www.npmjs.com/package/@types/relay-runtime

Thanks @mattkrick @alloy @sibelius

from relay-compiler-language-typescript.

Sure I can, but that's why I generate types. To use them.

from relay-compiler-language-typescript.

@mattkrick @alloy @sibelius Same for me. With @types/relay-runtime v1.3.8 works because of ConcreteRequest = any while with v1.3.10 breaks. I think that it is a problem with the typings because by semver, it is a breaking change ... not just a patch.
Could be possible to fix this issue:

• relax the types for ConcreteRequest in the typings
• fix this package to output compatible with v1.3.10
  ?

from relay-compiler-language-typescript.

you know, that's probably my bad, i made the typedef more strict to add support for getRequest.
relay v1.7.0 changed the defs a little bit, & now relay v2.0.0 has completely revamped the shape of some of these interfaces:

• old (v1.6-ish?): https://github.com/facebook/relay/blob/fe0e70f10bbcba1fff89911313ea69f24569464b/packages/relay-runtime/util/RelayConcreteNode.js
• new (v2.0.0): https://github.com/facebook/relay/blob/master/packages/relay-runtime/util/RelayConcreteNode.js

unless you're calling getRequest in your app, none of this really matters.

the right thing to do would be to release new typedefs for v.1.7.0 and another one for v2.0.0 (and my gut tells me that as @match matures, we'll need new defs for the next version, too)

the fast thing to do would be to excluded generated files from type checking, which should speed up build times, too

from relay-compiler-language-typescript.

ConcreteRequest will change very often, maybe it could be any or unknow, so it will break less

from relay-compiler-language-typescript.

we can relax ConcreteRequest types as they will keep changing and relay adds more transforms like @defer, @stream and so on

from relay-compiler-language-typescript.

@mattkrick is the "solution" to rollback this commit on typings?

DefinitelyTyped/DefinitelyTyped@f044c38#diff-03ed2ffcefe6022264efc0c0849fee14

I do believe that the compiler is a bit far from these typings.

from relay-compiler-language-typescript.

yeah, reverting for v1.3 is best. try keeping getRequest and just set it to any though.

I started v2 here: https://github.com/mattkrick/DefinitelyTyped/blob/v2.0/types/relay-runtime/index.d.ts
So much changed that I just started fresh. I ran flow-to-typescript over the whole thing & then made fixes where the conversion failed. still about 1 hour left to spend on it if anyone is itching to get it published feel free! (also i'm the worst with export syntax, feel free to let me know if there's a cleaner way!)

from relay-compiler-language-typescript.

Ok, closing this one. Looking forward to the typings for v2 and I expect to be able to pitch in on those again in another few weeks.

from relay-compiler-language-typescript.