search

meanjs / generator-meanjs

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

MEAN module subgenerator and test added #170

Closed UndeadBaneGitHub closed 8 years ago

codydaig commented 8 years ago

LGTM

codydaig commented 8 years ago

@ilanbiala LGTY?

UndeadBaneGitHub commented 8 years ago

Somehow, this branch started to have conflicts. I'm unsure, what is the conflict exactly, will fix it shortly.

ilanbiala commented 8 years ago

@UndeadBaneGitHub can you resolve the conflicts?

UndeadBaneGitHub commented 8 years ago

@ilanbiala yep, my bad. I will push it together with additional tests for routes.

ilanbiala commented 8 years ago

Awesome.

ilanbiala commented 8 years ago

Awesome.

tbaustin commented 8 years ago

How to create new module using cmd prompt?

UndeadBaneGitHub commented 8 years ago

@tbaustin You just call this generator and enter module name on prompt, then select the folders. However, it's not yet merged, need to fix the conflicts

tbaustin commented 8 years ago

@UndeadBaneGitHub I am typing in both yo meanjs:vertical-module and yo meanjs:crud-module and getting this same error for both. Is this a problem with the generator or something on my end?

error-> Error meanjs:vertical-module customers

You don't seem to have a generator with the name meanjs:vertical-module installed. You can see available generators with npm search yeoman-generator and then install them with npm install [name]. To see the 2 registered generators run yo with the --help option.

with crud getting same error just replace vertical with crud

ilanbiala commented 8 years ago

@tbaustin it isn't available to download yet. We'll post on the relevant issues once it is.

codydaig commented 8 years ago

@UndeadBaneGitHub @ilanbiala Is this one good to go?

UndeadBaneGitHub commented 8 years ago

@codydaig @ilanbiala Same here: all my commits are squashed into one, but I had to perform a merge operation afterwards, because of changes to gulpfile.js. I hope, that's ok with you.

codydaig commented 8 years ago

LGTM.

UndeadBaneGitHub commented 8 years ago

A new PR contains this generator now, with some adjustments and, of course, issues fixes: https://github.com/meanjs/generator-meanjs/pull/185. This one will be closed