ember-learn / cli-guides

Step-by-step guides and tutorials for using the ember-cli to create apps and addons
https://cli.emberjs.com
MIT License
25 stars 76 forks source link

add ember generate blueprints to cli reference #208

Closed neojp closed 3 years ago

neojp commented 3 years ago

This PR attempts to add additional CLI information for each ember generate blueprint provided by default. The blueprints are the following:

Related to #162


Screenshot

image

neojp commented 3 years ago

CI is failing with the following lint error, but I can't reproduce it in my local machine.

image

I guess we probably should ignore this file from eslint. Should I add it to the .eslintignore file?

maxwondercorn commented 3 years ago

@neojp That's an error from Alex which catches insensitive writing. You can override the error by putting this comment above the offending line

<!--alex ignore simple-->

neojp commented 3 years ago

@maxwondercorn thanks, that worked! How would you disable this retext-spell lint rule?

image

maxwondercorn commented 3 years ago

@neojp Normally the words would be added to the .local.dic file in the project root. They should not have to be added to the .local.dic file because they are surrounded by back-ticks.

I'm not sure why they are being flagged because ember-data is used elsewhere and it's not flagged.

If you've done an npm install, you can run the markdown linting locally with the npm run lint:md command. I'd be curious to see if the use of ember-data in the appendix/common-issues.md is flagged

BTW, nice job on the PR

neojp commented 3 years ago

I'm not sure why they are being flagged because ember-data is used elsewhere and it's not flagged.

@maxwondercorn Ah! What happened is that I added backticks to avoid getting these linter errors from showing up. I wanted to remove those backticks by adding an ignore rule, but I couldn't figure it out how. Thanks for the info, I've added the values to the .local.dic and it's working now.

All checks have passed. This should be ready to be merged now.