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: Do not add a newline before prompt when terminal opens #1016

Merged

Conversation

RhnSharma
Copy link
Contributor

Fixes #977
Hi @denysdovhan, how does this look?
Let me know if this needs any changes.
Thanks

@RhnSharma
Copy link
Contributor Author

@denysdovhan I have added a reference to the issue in a comment above the code. Is it correct?
Please take a look at it and let me know if this needs any changes.
Thanks

@denysdovhan denysdovhan added the improvement A PR that make small changes for improving UX, performance, readability, etc label Jun 17, 2021
@denysdovhan denysdovhan changed the title Fix : Do not add a newline before prompt when terminal opens Fix: Do not add a newline before prompt when terminal opens Jun 17, 2021
@denysdovhan denysdovhan merged commit eb6ed4d into spaceship-prompt:master Jun 17, 2021
@denysdovhan
Copy link
Member

Thanks for contributing!

denysdovhan added a commit that referenced this pull request Jun 17, 2021
* Do not add a newline before prompt when terminal opens

* Add a reference to the issue in a comment

* chore: format comment with a reference

Co-authored-by: Denys Dovhan <denysdovhan@gmail.com>
github-actions bot pushed a commit that referenced this pull request Jun 17, 2021
## [3.12.5](v3.12.4...v3.12.5) (2021-06-17)

### Bug Fixes

* Do not add a newline before prompt when terminal opens ([#1016](#1016)) ([aaf35d5](aaf35d5))
@Runrioter
Copy link
Member

@denysdovhan this makes terminal never show newline. Option SPACESHIP_PROMPT_ADD_NEWLINE does not work. I use iTerm on macOS Big Sur. Is this expected?

@denysdovhan
Copy link
Member

@Runrioter Nope. I have the same issue.

@denysdovhan
Copy link
Member

I have reverted these changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement A PR that make small changes for improving UX, performance, readability, etc
Development

Successfully merging this pull request may close these issues.

Do not add a new line before prompt when terminal opens
3 participants