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

Add space #1917

Closed
wants to merge 1 commit into from
Closed

Add space #1917

wants to merge 1 commit into from

Conversation

srifqi
Copy link
Member

@srifqi srifqi commented Dec 7, 2014

Add space between two online user:
What we want:

# Server: version=0.4.10, uptime=190.5, max_lag=0.001, clients={user1,user2,user3,user4,user5,foobar,userplayedminetestthismorning,thismorningiplayedminetestwithmyfriend}

Reality:

Server: version=0.4.10, uptime=190.5, max_lag=0.001, 
clients={user1,user2,user3,user4,user5,foobar,userplayedminetestthismorning,thismorningiplayed

Yes, the text is not complete.
So, I edit the file and it will show like this:

# Server: version=0.4.10, uptime=190.5, max_lag=0.001, clients={user1, user2, user3, user4,
user5, foobar, userplayedminetestthismorning, thismorningiplayedminetestwithmyfriend}

Also, this makes player to read it better.

Hope this helps!

Add space between two online user:
From:
# Server: version=0.4.10, uptime=190.5, max_lag=0.001, clients={user1,user2,user3,user4,user5,foobar,userplayedminetestthismorning}
To be:
# Server: version=0.4.10, uptime=190.5, max_lag=0.001, clients={user1, user2, user3, user4, user5, foobar, userplayedminetestthismorning}
@SmallJoker
Copy link
Member

Yeah, why not? 👍

@Zeno-
Copy link
Contributor

Zeno- commented Dec 7, 2014

I agree with this as well. A space after a comma is something most people expect.

@Calinou
Copy link
Member

Calinou commented Dec 7, 2014

I agree, it makes it more readable.

@Zeno-
Copy link
Contributor

Zeno- commented Dec 7, 2014

commit 5ffddc9

@srifqi
Copy link
Member Author

srifqi commented Dec 7, 2014

Thanks, hope this helps!

@srifqi srifqi deleted the patch-3 branch December 18, 2014 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants