search

moleculerjs / moleculer-cli

:herb: Command line tool for Moleculer
http://moleculer.services/docs/moleculer-cli.html
MIT License
48 stars 28 forks source link

feat: add mixin create #50

Closed devalexandre closed 2 years ago

devalexandre commented 2 years ago

Create Mixin generator command line

devalexandre commented 2 years ago

@icebob

devalexandre commented 2 years ago

@icebob was something missing?

icebob commented 2 years ago

I don't know, but it contains a lot of changes and I had no capacity to check it yet. Sorry, please be patient.

AndreMaz commented 2 years ago

I removed --detectOpenHandles --forceExit as these should not be necessary if all tests are properly implemented. After that I've executed npm test and got this image

devalexandre commented 2 years ago

I removed --detectOpenHandles --forceExit as these should not be necessary if all tests are properly implemented. After that I've executed npm test and got this image

I'm going to write this text, I think something is still asynchronous.

devalexandre commented 2 years ago

I removed --detectOpenHandles --forceExit as these should not be necessary if all tests are properly implemented. After that I've executed npm test and got this image

the test is not spelled wrong, but yargs is using promises, this is the test's problem, how to handle promises without forcing the output.

I'm researching about it, but I accept suggestions.

devalexandre commented 2 years ago

closed, to split the changes into small PRs.