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

Attempt at merging changes from upstream #3

Open
wants to merge 2 commits into
base: LVTProcessing
Choose a base branch
from

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" [email protected] 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