Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: no more yeoman #519

Merged
merged 6 commits into from
Jul 29, 2024
Merged

feat: no more yeoman #519

merged 6 commits into from
Jul 29, 2024

Conversation

mdonnalley
Copy link
Collaborator

@mdonnalley mdonnalley commented Jul 26, 2024

What does this PR do?

  • Remove yeoman as a dependency (affects dev generate command|library|plugin)
  • Add --dry-run flag to dev generate command|library|plugin

What issues does this PR fix or reference?

@W-16025394@

@mdonnalley mdonnalley requested a review from a team as a code owner July 26, 2024 21:30
}

this.ux.log(`${updating} package.json`);
await writeFile(join(this.cwd, 'package.json'), JSON.stringify(this.pjson, null, 2));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we had to normalize this path to fix a bug recently might need to be carried over here too

@WillieRuemmele
Copy link
Contributor

QA Notes


✅ : no yeoman

➜  plugin-dev git:(mdonnalley/no-yeoman) 
 ➜  yarn why yeoman                  
yarn why v1.22.22
[1/4] 🤔  Why do we have the module "yeoman"...?
[2/4] 🚚  Initialising dependency graph...
[3/4] 🔍  Finding dependency...
error We couldn't find a match!
✨  Done in 0.24s.

dev generate plugin
✅ : package name validation
✅ : creates plugin
✅ : --dry-run no changes, prints what would change
✅ : installs/builds command

 ➜  testing/bin/run.js hello:world
Hello World at Mon Jul 29 2024.

✅ : internal plugin correct


dev generate command

✅ : plugin checks performed and passed


dev generate library

✅ : previous checks

@WillieRuemmele WillieRuemmele merged commit c7ac1da into main Jul 29, 2024
13 checks passed
@WillieRuemmele WillieRuemmele deleted the mdonnalley/no-yeoman branch July 29, 2024 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants