load collection path from nest-cli.json

[taozhiyuzhuo]

I'm submitting a...

[ ] Regression 
[ ] Bug report
[x ] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

Current behavior

collection default value is @nestjs/schematics. if we want to use custom schematics. I must execute nest g -c custom/schematics

Expected behavior

• nest cli can load custom schematics path from nest-cli.json. if I set the "collection": "@custom/schematics"
  when I execute nest g it can load my custom schematics
• it should be better if it can merge my custom schematics to @nestjs/schematics.

What is the motivation / use case for changing the behavior?

Generate files by custom schematics more convenient

[kamilmysliwiec]

nest cli can load custom schematics path from nest-cli.json. if I set the "collection": "@custom/schematics"
when I execute nest g it can load my custom schematics

Setting "collection": "@custom/schematics". Does it not work for you?

[taozhiyuzhuo]

nest cli can load custom schematics path from nest-cli.json. if I set the "collection": "@custom/schematics"
when I execute nest g it can load my custom schematics

Setting "collection": "@custom/schematics". Does it not work for you?

yes, it does not work. always use @nestjs/schematics

[kamilmysliwiec]

Would you like to create a PR for this issue?

[taozhiyuzhuo]

Would you like to create a PR for this issue?

sorry for the response you late. is that PR ok?

[rthomps7]

@kamilmysliwiec

I don't think this PR is really what you need because t makes it so the nest-cli.json configuration will always override the CLI. What we realy need is the Collection.NESTJS fallback here to pull from the nest-cli.json

[kamilmysliwiec]

@rthomps7 would you like to create a PR with a fix?

[ledniy]

Hi, @kamilmysliwiec could you review PR #1166 that addressing that issue? Generally, I've used @rthomps7 suggestion and it worked.

[kamilmysliwiec]

Let's track this here #1166