SAP / generator-easy-ui5

Meta-generator various project types within the UI5 Universe
https://blogs.sap.com/2021/04/09/easy-ui5-3.0-from-community-contributions-to-community-plugins/
Apache License 2.0
235 stars 72 forks source link

fix: disable autoinstall of Yeoman 5.x #82

Closed petermuessig closed 2 years ago

IObert commented 2 years ago

Thanks Peter. Does this change also affect the other plugins such as generator-ui5-project, generator-ui5-library etc?

petermuessig commented 2 years ago

Every generator now runs this install feature at the end (this is the change of Yeoman 5.x). This fixes mainly the first message:

> ~  % yo easy-ui5

     _-----_     
    |       |    ╭──────────────────────────╮
    |--(o)--|    │  Welcome to the easy-ui5 │
   `---------´   │        generator!        │
    ( _´U`_ )    ╰──────────────────────────╯
    /___A___\   /
     |  ~  |     
   __'.___.'__   
 ´   `  |° ´ Y ` 

? Select your generator? generator-ui5-ts-app
? How do you want to name this application? (myapp) 
No change to package.json was detected. No package manager install will be executed.

No change to package.json was detected. No package manager install will be executed.

This fix avoids this message which is typically logged out after the first question of the sub generator appears and confuses the people. So, yes - it affects all sub generators - but every sub generator should also disable the install of Yeoman 5.x as long as they handle their own install. The good thing: as long as the sub generators stay on Yeoman 4.x there is no problem. In addition, I have to admit, I also don't like the many logs the new install feature of Yeoman 5.x creates - this is annoying and this install feature is not perfectly thought through IMO.

IObert commented 2 years ago

So, yes - it affects all sub generators

So let's include all the other maintainers here: @vobu @ubheamar @geert-janklaps

IObert commented 2 years ago

@petermuessig I understand most of the diff in this PR. But I don't understand why you replaced this line with a comment. Is this also related?

this.config.set("setupCompleted", true);
ubheamar commented 2 years ago

So, yes - it affects all sub generators

So let's include all the other maintainers here: @vobu @ubheamar @geert-janklaps

I have made changes in FLP Plugin Repository

petermuessig commented 2 years ago

@petermuessig I understand most of the diff in this PR. But I don't understand why you replaced this line with a comment. Is this also related?

this.config.set("setupCompleted", true);

Fixed with: https://github.com/ui5-community/generator-ui5-ts-app/commit/03e8d81b261a83be3d77d46d72a16770f7a83f10

vobu commented 2 years ago

So, yes - it affects all sub generators

So let's include all the other maintainers here: @vobu @ubheamar @geert-janklaps

generator-ui5-project (which i don't feel to be the maintainer of, but rather a maintainer of 😸) for now stays on yeoman 4.x - so as Peter describes, for the time being no issue here. But: I'll create an issue in generator-ui5-project to make the community aware of this necessary change when switching to yeoman 5

IObert commented 2 years ago

Thanks