Skip to content
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

Fix #414 #482

Merged
merged 1 commit into from Jan 26, 2014
Merged

Fix #414 #482

merged 1 commit into from Jan 26, 2014

Conversation

jdespatis
Copy link
Contributor

Go back to previous behavior, in order to have:

     info This might take a few moments
....
✔ Done in /home/jerome/.cache/yeoman/symfony/symfony-standard/v2.4.1

Go back to previous behavior, in order to have:
```
     info This might take a few moments
....
✔ Done in /home/jerome/.cache/yeoman/symfony/symfony-standard/v2.4.1
```
@sindresorhus
Copy link
Member

I think this should be solved by adding a newline after ...., not before .

SBoudrias added a commit that referenced this pull request Jan 26, 2014
@SBoudrias SBoudrias merged commit 116b321 into yeoman:master Jan 26, 2014
@SBoudrias
Copy link
Member

As the dots . are printed on data events. The place to insert the line feed is in the complete events. This seems allright to me.

@sindresorhus
Copy link
Member

@SBoudrias i don't like that we have to care about the dots being precedent. Can't you just print and '\n' on the end event after data events have occurred?

@SBoudrias
Copy link
Member

@sindresorhus Isn't this what is happening here?

@sindresorhus
Copy link
Member

@SBoudrias yes, but we do it explicitly. It should just happen.

i don't like that we have to care about the dots being precedent

We have to add the extra .write() because the dots is precedent.

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.

None yet

3 participants