Skip to content

Compact layout#124

Merged
ntwcklng merged 2 commits intoleo:masterfrom
oskarrough:compact-layout
May 23, 2018
Merged

Compact layout#124
ntwcklng merged 2 commits intoleo:masterfrom
oskarrough:compact-layout

Conversation

@oskarrough
Copy link
Copy Markdown
Contributor

This PR makes the default output take up less space in the terminal by removing some extra line breaks. Here's a before/after:

before after

Ideally we would also remove the extra two line breaks at the end but I could not figure out how to remove them?

Closes #123.

@oskarrough oskarrough mentioned this pull request May 22, 2018
@ntwcklng
Copy link
Copy Markdown
Collaborator

Looks good! There is only 1 extra line at the end, the other one comes from your terminal i think. This is the output on Codeanywhere:
unbenannt

Many thanks!

@ntwcklng ntwcklng merged commit 4a474a0 into leo:master May 23, 2018
@oskarrough
Copy link
Copy Markdown
Contributor Author

@ntwcklng do you think we could have a new NPM release including this?

@ntwcklng
Copy link
Copy Markdown
Collaborator

@oskarrough released in 5.0.0 :shipit:

joshbetz added a commit to Automattic/vip-cli that referenced this pull request Jun 21, 2018
joshbetz added a commit to Automattic/vip-cli that referenced this pull request Jun 22, 2018
leo pushed a commit that referenced this pull request May 1, 2022
* Add example examples to the test

* Remove extra line breaks in output
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