Skip to content

Conversation

@benkobalog
Copy link
Contributor

No description provided.

Copy link
Member

@dottybot dottybot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, and thank you for opening this PR! 🎉

All contributors have signed the CLA, thank you! ❤️

Have an awesome day! ☀️

@smarter
Copy link
Member

smarter commented May 8, 2018

It looks like you accidentally changed the scala-backend submodule in your commit.

@benkobalog
Copy link
Contributor Author

Yes, I was looking at that too. I'm not sure what happened there. Should I just revert it or make a new PR?

@smarter
Copy link
Member

smarter commented May 8, 2018

You can amend the commit and push-force.

else if (numberStart(n)) {
def isNumber(c: Char): Boolean =
c.isDigit || c == '\u0000' || (c == '.' && remaining.nonEmpty && remaining.head.isDigit)
appendWhile(n, isNumber , literal)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there's an extra space after isNumber that shouldn't be there

@benkobalog
Copy link
Contributor Author

Thanks for the quick fix @Blaisorblade ! Is there some code formatter setting for the code base or how do you format the code usually (I haven't found anything about this)?

@smarter
Copy link
Member

smarter commented May 8, 2018

There isn't anything automatic currently, just make sure the code you're writing looks like it fits with the code around it.

@smarter smarter merged commit 40258da into scala:master May 8, 2018
@benkobalog benkobalog deleted the fix-#4454 branch May 10, 2018 09:15
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.

4 participants