Skip to content

Update MinecraftCom.cs#8

Closed
dogwatch wants to merge 1 commit intoMCCTeam:Indevfrom
dogwatch:patch-2
Closed

Update MinecraftCom.cs#8
dogwatch wants to merge 1 commit intoMCCTeam:Indevfrom
dogwatch:patch-2

Conversation

@dogwatch
Copy link
Contributor

No description provided.

ORelio added a commit that referenced this pull request Jul 20, 2013
Changes could not be merged automatically due to a conflic.
#8
So I re-do the changes manually and close the original pull request.
@ORelio
Copy link
Member

ORelio commented Jul 20, 2013

"We can’t automatically merge this pull request." -> Some conflicts have to be resolved before continuing.
Manually added your changes in this commit: de50e07 so I close the pull request but your changes have actually been added to the project, thanks!

NB: Not sure of the usefulness of using String.Empty over simply writing ""
More info at http://stackoverflow.com/questions/263191/in-c-should-i-use-string-empty-or-string-empty-or
And at http://www.codinghorror.com/blog/2007/05/the-best-code-is-no-code-at-all.html

@ORelio ORelio closed this Jul 20, 2013
milutinke added a commit that referenced this pull request Dec 15, 2023
milutinke added a commit that referenced this pull request Mar 22, 2026
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.

2 participants