Issue migrating to TypeScript about generator-license HOT 10 CLOSED

@jozefizso I completely understand your point and in that case I would suggest I will just maintain a TypeScript fork so that I don't disturb your work. @mischah I will then update the dependencies to point to a package like @groovytron/generator-license in generator-license if that's fine for you. So that we have a solution that satisfies everybody.

And I'll see if I can move this issue to my fork. Otherwise I'll just copy it into mine.

Is that fine for everybody ?

Thanks for your inputs.

from generator-license.

I checked the https://github.com/yeoman/generator-generator source code and I don't see any reference to to this package there.

Just to let you know:
It’s a transitive dependency. Buried not that deep in the dependency tree:
generator-generator → generator-node → generator-license

Regarding TypeScript in the Yeoman generator eco system.
Using Typescript is not necessary at all. But it’s so damn comfortable to have a much more useful autocompletion in your Editor. So people using your generator when composing their generators would benefit from that. And yes, the additional built step sucks.

Chhers, Michael

from generator-license.

What issue does TypeScript solve here? The app and code is stable for years and well tested.

from generator-license.

I'm trying to upgrade yeoman/generator-generator (cf. this issue for more details) to use the latest version of yeoman-generator (which is 5.7.0 at the time of this writing).

As yeoman/generator-generator depends on generator-licenses, I started to try to migrate generator-license to TypeScript first.

The end result should be the same. There is no change on the end user point of view. It's just a dependencies upgrade.

And the tests remains the same which means nothing that was tested in the current version will break in the next version.

I hope it helps to understand the idea of this migration. I don't want to enforce it. I can still migrate my own fork if migrating to TypeScript is a no go.

from generator-license.

I checked the https://github.com/yeoman/generator-generator source code and I don't see any reference to to this package there.

If this package is to be used from TypeScript code a type definition file can be created. I don't see a point in rewriting this app and make it much more complicated with additional tools for compilation when it is working fine.

from generator-license.

So it's for these three lines https://github.com/yeoman/generator-node/blob/25d1ca4ea2122ce08b88398848e6df41f54cbe1a/generators/app/index.js#L363-L366

That can be done by type definition and not by requiring the whole project to be change to a different toolset.

from generator-license.

And the intellisense is already available:

const lic = require('generator-license');
const x = lic.licenses;

from generator-license.

A fork doesn’t make sense in this case. Since This project is maintained. Why don’t you use this as it is? And create a PR adding type definitions in case @jozefizso will accept to merge this.

What’s your pain with having a JavaScript dependency in a TypeScript Repo?

from generator-license.

I will accept TypeScript type definitions for this repo. The change to TypeScript tooling is our of question.

from generator-license.

I am really sorry. I didn't understand that having the types definition would actually solve the issue. Thanks for the explaination

I will then add the types definitions and open a PR for this. Thanks for the help and have both a nice weekend.

from generator-license.