Skip to content

Conversation

tiennou
Copy link
Contributor

@tiennou tiennou commented Feb 16, 2017

No description provided.

@ssp
Copy link
Contributor

ssp commented Feb 16, 2017

I like it … I’d just like to get #30 and #35 in first as they modify xibs – and I suspect we may end up with a merge conflict if the files are moved away first.

@tiennou
Copy link
Contributor Author

tiennou commented Feb 16, 2017

I don't think it will, it's just a rename and git handles that just fine. I actually merged both of those into this one without issue (apart from the fact that submodules are really funny).

BTW, you might have to git submodule deinit --all/git submodule init --update --recursive at least once before the dust settles.

@tiennou tiennou closed this Feb 16, 2017
@tiennou tiennou reopened this Feb 16, 2017
@tiennou tiennou force-pushed the shuffle-things branch 2 times, most recently from ed0e214 to a6561fd Compare February 16, 2017 22:43
@tiennou
Copy link
Contributor Author

tiennou commented Feb 20, 2017

Merging in ! For the record, I have pushed a stage-menu-shuffled branch in my repo (in case it's easier / for reference).

@tiennou tiennou merged commit 28ad8d9 into gitx:master Feb 20, 2017
@tiennou tiennou deleted the shuffle-things branch February 26, 2017 22:05
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.

2 participants