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

Guild refactor #11

Closed
wants to merge 12 commits into from
Closed

Guild refactor #11

wants to merge 12 commits into from

Conversation

craineum
Copy link
Collaborator

Removed documentation so we can clean it up later. Moved a spec that was out of place for hot. Added objects for guild requests and results.

@jemiahlee
Copy link
Owner

please rebase this so that I can review it a little easier. :) thanks!

@craineum
Copy link
Collaborator Author

Had issues with the rebase, so ended up doing a merge, but the overall results remain the same on what has changed. Sorry about that.

@jemiahlee
Copy link
Owner

Well, the difference between merge and rebase is in the fact that other people may be using this as well, and merges will complicate their ability to also submit pull requests. Which is an unfriendly way to work on an open-source project.

@jemiahlee
Copy link
Owner

I'll look at your branch and try to figure out what it is that you did that complicated the rebase. It's probably that there is another merge in here other than the PR that I merged.

@jemiahlee
Copy link
Owner

And I realize it's academic on this, since there is almost certainly no one else working on the project, but in general on OSS projects, you'll want to make sure you never merge.

@craineum
Copy link
Collaborator Author

No, I completely agree about it. I am learning this, since this is the first OSS I have contributed to. I am lucky to have you helping me break my teeth in on it.

What I did...

git fetch upsteam
git checkout origin master
git rebase upstream/master
git push origin master
git checkout origin guild_refactor
git rebase origin/master

This is where it gave me a "you local has diverged from origin/guild_refactor"

Told me to correct by doing a pull

git pull origin guild_refactor
git push origin guild_refactor

@jemiahlee jemiahlee mentioned this pull request Sep 3, 2014
@craineum
Copy link
Collaborator Author

Closing this since it was merged via another pull request from @jemiahlee

@craineum craineum closed this Oct 23, 2014
@craineum craineum deleted the guild_refactor branch October 23, 2014 17:12
@jemiahlee
Copy link
Owner

All the commits are yours, though! :)

@craineum
Copy link
Collaborator Author

Haha... true :) I am getting back into this, and was just doing some clean-up. Sorry it has taken me so long!

@jemiahlee
Copy link
Owner

No worries! I've been swamped, I'm managing two teams at work.

Are you going to BGG?

@craineum
Copy link
Collaborator Author

No, Sold out by the time I went to get tickets :(

@jemiahlee
Copy link
Owner

:-( Oh well, I was hoping to meet you! Maybe next year?

@craineum
Copy link
Collaborator Author

Yes, I plan on going to more board game conventions in the coming year. I have a couple of games I am working on myself.

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