romainberger / yeoman-flask

Yeoman generator for Flask project
MIT License
73 stars 13 forks source link

add basic testing to generator #16

Closed idbentley closed 10 years ago

idbentley commented 10 years ago

Need to rename the main Flask object to app to avoid naming conflicts.

romainberger commented 10 years ago

Hi, thanks for all this PR!

I've merged some of them but would like to try the two remaining before merging. I can't do it right now, will try to get back asap.

idbentley commented 10 years ago

Sounds good. I had to rename the main app object away from appname because I couldn't figure out how to avoid import shadowing - maybe you can. I didn't see any harm in calling it app, and it more closely matches the Flask documentation.

romainberger commented 10 years ago

Thanks a bunch! :+1: