Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upIs UnityYAMLMerge actually being used? #943
Comments
|
Hey @Piotr-B. Honestly, thanks for your analysis. I will try next week to investigate and resolve the issues your raised. |
|
Hi guys, I would like to start using GitHub for Unity for our next project, and this bug sounds pretty important to me. At what point you honestly are with this? Thanks, |
|
Hey @fabiopolimeni thanks for checking our stuff out. |
Description
While investigating using this plugin for our next project I noticed that the UnityYAMLMerge tool seems to get configured for use but never actually gets launched.
When creating a project with the plugin the following configuration options get set:
.gitattributes:
.git/config:
This looks well and good but:
mergeattribute looks for a custom low-level merge driver in the config file in the following format:[merge "unityyamlmerge"]but external merge tool fields:cmd =trustExitCode =. These are different things but appear to be mixed here. Amergetoolsection is supposed to look like this:But for that to work it also requires this entry, which is missing:
And I think that would launch the tool for all file types not just the select ones.
Disclaimer:
I'm not very proficient in git so I might be missing something here but this is what it looks like to me after investigating the code and docs.
I'm using GitHub for Unity 1.1.0 and Unity 2018.1.1f1.
References
https://git-scm.com/book/en/v2/Customizing-Git-Git-Configuration#_external_merge_tools
https://git-scm.com/docs/gitattributes#_defining_a_custom_merge_driver
https://git-scm.com/docs/git-config#git-config-mergetool
https://docs.unity3d.com/Manual/SmartMerge.html
Unity/src/GitHub.Api/Resources/.gitattributes
Lines 3 to 7 in 1c53148
Unity/src/GitHub.Api/Application/ApplicationManagerBase.cs
Lines 291 to 299 in 1c53148