New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

padrino-gen app should create a Rakefile #925

Closed
postmodern opened this Issue Aug 29, 2012 · 8 comments

Comments

Projects
None yet
5 participants
@postmodern
Contributor

postmodern commented Aug 29, 2012

Other rake tasks, such as resque/tasks, do not work with padrino rake and requires padrino rake gen be ran. There doesn't appear to be any benefit in not creating a dummy Rakefile.

@nesquena

This comment has been minimized.

Show comment
Hide comment
@nesquena

nesquena Sep 23, 2012

Member

Good point, I think we an have a Rakefile generated by default in the next release? I always end up generating it anyways @DAddYE @achiu

Member

nesquena commented Sep 23, 2012

Good point, I think we an have a Rakefile generated by default in the next release? I always end up generating it anyways @DAddYE @achiu

@achiu

This comment has been minimized.

Show comment
Hide comment
@achiu

achiu Sep 23, 2012

Member

@nesquena yeah, I agree with this. I'm for an explicit Rakefile.

Member

achiu commented Sep 23, 2012

@nesquena yeah, I agree with this. I'm for an explicit Rakefile.

@DAddYE

This comment has been minimized.

Show comment
Hide comment
@DAddYE

DAddYE Sep 23, 2012

Member

okyz!

Member

DAddYE commented Sep 23, 2012

okyz!

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Nov 28, 2012

yup, I am also manually generating it in nearly any case

ghost commented Nov 28, 2012

yup, I am also manually generating it in nearly any case

@ghost ghost assigned achiu Dec 29, 2012

@achiu achiu closed this in 4fd1cc3 Dec 29, 2012

@nesquena

This comment has been minimized.

Show comment
Hide comment
@nesquena

nesquena Dec 30, 2012

Member

Sweet thanks @achiu

Member

nesquena commented Dec 30, 2012

Sweet thanks @achiu

@ujifgc

This comment has been minimized.

Show comment
Hide comment
@ujifgc

ujifgc Dec 30, 2012

Member

Excellent! Fix enough for #971

Member

ujifgc commented Dec 30, 2012

Excellent! Fix enough for #971

@ujifgc

This comment has been minimized.

Show comment
Hide comment
@ujifgc

ujifgc Dec 30, 2012

Member

There's a typo in the name of the template. Should be Rakefile.tt

Member

ujifgc commented Dec 30, 2012

There's a typo in the name of the template. Should be Rakefile.tt

@achiu

This comment has been minimized.

Show comment
Hide comment
@achiu

achiu Dec 30, 2012

Member

@ujifgc ah thanks. should have written better tests to check myself 😦

Member

achiu commented Dec 30, 2012

@ujifgc ah thanks. should have written better tests to check myself 😦

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment