Skip to content
This repository has been archived by the owner on Nov 27, 2018. It is now read-only.

clarify output of deploy command #163

Closed
mykmelez opened this issue Nov 10, 2015 · 1 comment
Closed

clarify output of deploy command #163

mykmelez opened this issue Nov 10, 2015 · 1 comment
Assignees
Milestone

Comments

@mykmelez
Copy link
Contributor

The deploy command's output could be clearer. Here's an example of the current design:

[16:21:44] Deploying "ignore cache created when using oghliner CLI"
[16:21:44] Cloning git@github.com:mykmelez/eggtimer.git into .gh-pages-cache
[16:21:44] Cleaning
[16:21:44] Fetching origin
[16:21:45] Checking out origin/gh-pages
[16:21:45] Removing files
[16:21:45] Copying files
[16:21:45] Adding all
[16:21:45] Committing
[16:21:45] Pushing

As with #162, this is basically fine, modulo unnecessary timestamps. Perhaps we'd want to add an ellipsis (…) to the end of each step, then prepend a green checkmark and append the word "done!" after the step is complete, like for configure in #148.

I'd also update the header line to look like:

Deploying "ignore cache created when using oghliner CLI" to GitHub Pages…

And if the commit message is multi-line, I'd ensure that only the first line appears in that header line.

As with #162, this command runs frequently and often is automated as part of a build process, so we should avoid extraneous header/footer information.

@marco-c
Copy link
Contributor

marco-c commented Nov 11, 2015

#172 is removing the timestamps.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants