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

Don't change \n to an actual newline #29

Closed
aboyton opened this issue Feb 14, 2018 · 8 comments
Closed

Don't change \n to an actual newline #29

aboyton opened this issue Feb 14, 2018 · 8 comments
Assignees
Labels

Comments

@aboyton
Copy link

aboyton commented Feb 14, 2018

First up, awesome work on this project, I'm really excited. I seem to recall there was some discussion about this for JavaScript but I can't find the issue.

Right now this converts

$foo = "hello\nworld"

into

$foo = "hello
world"

Personally I think it is nicer to leave the \n alone.

@mgrip
Copy link
Contributor

mgrip commented Feb 14, 2018

🙏 Thanks for finding these issues @aboyton! Keep em coming 😉

@aboyton
Copy link
Author

aboyton commented Feb 14, 2018

No worries. Not sure how much time I'll have to help with the coding, but very happy to help with the testing. I've got a reasonably large PHP codebase to run this on.

@azz azz added the bug label Feb 17, 2018
@alexander-akait
Copy link
Member

WIP

@alexander-akait
Copy link
Member

@mgrip sorry, did not have time today, tomorrow I will do

@mgrip
Copy link
Contributor

mgrip commented Feb 27, 2018

No worries, thanks!

@mgrip
Copy link
Contributor

mgrip commented Mar 1, 2018

@evilebottnawi fixed with #136

@mgrip mgrip closed this as completed Mar 1, 2018
@aboyton
Copy link
Author

aboyton commented Mar 25, 2018

This still appears to be happening for me in certain cases, in particular if I have a variable directly in a string like below. I'm happy to open a new issue if you'd like.

printf("Query run in $queryTimeTaken seconds.\n");

This was referenced Mar 25, 2018
@czosel czosel reopened this Mar 26, 2018
@mgrip mgrip self-assigned this Mar 29, 2018
@alexander-akait
Copy link
Member

Done in #263

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

No branches or pull requests

5 participants