Skip to content
This repository was archived by the owner on Dec 12, 2023. It is now read-only.

Conversation

Widea
Copy link
Contributor

@Widea Widea commented Aug 9, 2015

Added alternate solution to CombineTwoStrings.java

@blakeembrey
Copy link
Owner

Thanks for the contributions 👍 I'm going to merge this now, sorry it took a couple of days to get back to you.

Side note: You should check out how to rebase/amend commits with git, makes for a much nicer history instead of having a bunch of "fix" commits. Another thing which a lot of projects will enforce is consistent style (E.g. space between if and ().

blakeembrey added a commit that referenced this pull request Aug 11, 2015
updated BinarySearchTree.java
@blakeembrey blakeembrey merged commit de315a5 into blakeembrey:master Aug 11, 2015
@Widea
Copy link
Contributor Author

Widea commented Aug 12, 2015

Thanks for the response!

The comments really help. I will look into it. Thank you.

@Widea Widea deleted the widea-local branch August 12, 2015 01:14
blakeembrey added a commit that referenced this pull request Dec 21, 2015
updated BinarySearchTree.java
blakeembrey added a commit that referenced this pull request Sep 29, 2017
updated BinarySearchTree.java
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants