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

Conversation

@emckee2006
Copy link

Hopefully I did this correctly. Feel free to give me some feedback...

emckee2006 and others added 2 commits September 18, 2015 12:06
More tweaks to merging. It's pretty decent. But still no where near g…
@LexManos
Copy link
Member

No, its done as a sub-module split so that we can do a merge where it keeps all of the history correctly as commits.
So no this is not done correctly.
However, we're not particularly worried about merging other people's work until we get the LVT processing done. Once the LVT branch is done we'll cleanup the community history, sync master and pull down incoming changes.
But LVT stuff is the most important part right now.

@emckee2006
Copy link
Author

Sorry about that....what's the right thing to do, then?
On Sep 18, 2015 5:10 PM, "LexManos" notifications@github.com wrote:

No, its done as a sub-module split so that we can do a merge where it
keeps all of the history correctly as commits.
So no this is not done correctly.
However, we're not particularly worried about merging other people's work
until we get the LVT processing done. Once the LVT branch is done we'll
cleanup the community history, sync master and pull down incoming changes.
But LVT stuff is the most important part right now.


Reply to this email directly or view it on GitHub
MinecraftForge/FernFlower#3 (comment)
.

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