search

meanjs / generator-meanjs

MEAN.JS Official Yeoman Generator
http://meanjs.org/
472 stars 178 forks source link

$ yo meanjs:angular-module <module-name> - Docs on the site are out of date #208

Open rohanleach4 opened 8 years ago

rohanleach4 commented 8 years ago

$ yo meanjs:angular-module

no longer is correct. I believe it should be:

$ yo meanjs:mean-module

simison commented 8 years ago

Wanna send a PR? :-)

rohanleach4 commented 8 years ago

I would love to, but in this case, I am not sure how to :-/

On 19 March 2016 at 11:14, Mikael Korpela notifications@github.com wrote:

Wanna send a PR? :-)

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/meanjs/generator-meanjs/issues/208#issuecomment-198685602

nodox commented 8 years ago

I just ran into this problem. Would love to have updated docs. @rohanleach4 do you need help creating a pull request?

rohanleach4 commented 8 years ago

@nodox yes please. That would be a great help

nodox commented 8 years ago

@rohanleach4 Here is a great tutorial on how to create PR

tl;dr

  1. Create local copy,
  2. pull from origin,
  3. create a local branch for the fix
  4. code, commit, push branch with fix to origin
  5. create PR on PR page with your branch.

The first half is all you really need but the bits towards the end are useful once you start contributing more.

rohanleach4 commented 8 years ago

Sorry for my tardiness of reply - stacked out at the moment. I will look into this over this coming week - thanks for your time :-)