Skip to content

Logs commands executed and print Done when finished #17

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

Merged
merged 2 commits into from
Mar 22, 2019

Conversation

localhostdotdev
Copy link

Would this be a good middle instead of logging the full output of the commands (#12) it logs the command executed and Done when finished (I'm actually not sure about the "Done" because there is just going to be more logs after so "Done" is already implicit).

else
log "\"#{command}\" failed to execute"
exit
end
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could probably be included in the Running module with env_as_assigns there too.

@localhostdotdev localhostdotdev changed the title log command executed and Done when finished Logs commands executed and print Done when finished Mar 21, 2019
@localhostdotdev
Copy link
Author

Handling of errors could be a separate PR but I'm trying not the spam too much. e.g. I'm getting "rvm 2.5.3 do bundle _1.16.1_ install --without db test job cable storage ujs >/dev/null" failed to execute, need to debug that.

@rafaelfranca rafaelfranca merged commit 98fb2c0 into rails:master Mar 22, 2019
@localhostdotdev localhostdotdev deleted the log-and-done branch March 22, 2019 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants