Skip to content

Conversation

@notmessenger
Copy link
Contributor

I know that these instructions can be derived/inferred from the "Generating an in-repo-addon blueprint" section but when scanning the headings it took several passes over the content before I made this realization. Thought maybe a specific section highlighting this usage would be beneficial.

@netlify
Copy link

netlify bot commented Oct 27, 2021

✔️ Deploy Preview for ember-cli-guides ready!

🔨 Explore the source changes: 2a79f81

🔍 Inspect the deploy log: https://app.netlify.com/sites/ember-cli-guides/deploys/61a8d25d9d12960007af6815

😎 Browse the preview: https://deploy-preview-239--ember-cli-guides.netlify.app

Copy link
Contributor

@jenweber jenweber left a comment

Choose a reason for hiding this comment

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

Thank you for this improvement! I have a couple suggestions. After applying the one using the full generate command, I think this is good to merge.

@jenweber jenweber self-requested a review December 2, 2021 14:04
Copy link
Contributor

@jenweber jenweber left a comment

Choose a reason for hiding this comment

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

Thank you!

@jenweber jenweber enabled auto-merge December 2, 2021 14:04
@jenweber jenweber disabled auto-merge December 2, 2021 14:08
@jenweber jenweber merged commit 7d6fa24 into ember-learn:main Dec 2, 2021
@notmessenger notmessenger deleted the generators branch December 2, 2021 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants