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

Add CLI console 'runner' command for run a piece of code #927

Merged
merged 3 commits into from Sep 8, 2012

Conversation

Projects
None yet
5 participants
@tyabe
Contributor

tyabe commented Sep 6, 2012

'runner' command is run Ruby code in the context of Padrino.
You can also be Alternatively 'run' or 'r'.

@nesquena

This comment has been minimized.

Show comment
Hide comment
@nesquena

nesquena Sep 6, 2012

Member

Thanks for the pull request. I could see the benefit of having a 'runner', what do you think @DAddYE

Member

nesquena commented Sep 6, 2012

Thanks for the pull request. I could see the benefit of having a 'runner', what do you think @DAddYE

@DAddYE

This comment has been minimized.

Show comment
Hide comment
@DAddYE

DAddYE Sep 6, 2012

Member

@tyabe we need also --chroot and if you can try to follow the code guidelines of other tasks. Btw I really wanted it!

BUT, please check always tests.

Thanks!

Member

DAddYE commented Sep 6, 2012

@tyabe we need also --chroot and if you can try to follow the code guidelines of other tasks. Btw I really wanted it!

BUT, please check always tests.

Thanks!

@pepe

This comment has been minimized.

Show comment
Hide comment
@pepe

pepe Sep 6, 2012

Contributor

+1

Contributor

pepe commented Sep 6, 2012

+1

@tyabe

This comment has been minimized.

Show comment
Hide comment
@tyabe

tyabe Sep 6, 2012

Contributor

Hi @DAddYE. I tried '-c' and '-e' options with 'runner' command.
Please let me know if there is a guideline to follow:)

Contributor

tyabe commented Sep 6, 2012

Hi @DAddYE. I tried '-c' and '-e' options with 'runner' command.
Please let me know if there is a guideline to follow:)

@dariocravero

This comment has been minimized.

Show comment
Hide comment
@dariocravero

dariocravero Sep 8, 2012

Contributor

+1

Contributor

dariocravero commented Sep 8, 2012

+1

DAddYE added a commit that referenced this pull request Sep 8, 2012

Merge pull request #927 from tyabe/add_runner
Add CLI console 'runner' command for run a piece of code

@DAddYE DAddYE merged commit df97bd1 into padrino:master Sep 8, 2012

1 check passed

default The Travis build passed
Details
@DAddYE

This comment has been minimized.

Show comment
Hide comment
@DAddYE

DAddYE Sep 8, 2012

Member

Thanks man!

Member

DAddYE commented Sep 8, 2012

Thanks man!

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