Remove US_Campaign-TEST.conf that duplicates the live config's GUID#91
Open
Smiggus wants to merge 1 commit into
Open
Remove US_Campaign-TEST.conf that duplicates the live config's GUID#91Smiggus wants to merge 1 commit into
Smiggus wants to merge 1 commit into
Conversation
US_Campaign-TEST.conf was copied from US_Campaign.conf together with its .meta file, so both resources in 7CavPVE claim GUID ADFDBDA163950168. Duplicate GUIDs inside one addon make resource resolution ambiguous: whichever file the engine indexes last wins, so the live US_Campaign faction config can silently be replaced by the TEST copy (they differ by ~300 lines). Nothing references the TEST file by path and no other file references the GUID, so remove it. If the TEST variant is still wanted, it should be re-created through Workbench so it receives its own GUID.
Smiggus
force-pushed
the
fix/remove-duplicate-guid-test-config
branch
from
47b7cb2 to
1626858
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
US_Campaign-TEST.confwas copied fromUS_Campaign.conftogether with its.meta, so both resources in 7CavPVE claim GUIDADFDBDA163950168. Duplicate GUIDs in one addon make resolution ambiguous: whichever file the engine indexes last wins, so the live US faction config can silently be replaced by the TEST copy (they differ by roughly 300 lines).Nothing references the TEST file by path and nothing references the GUID other than users of the live config, so this removes the TEST copy. If the TEST variant is still needed, re-create it through Workbench so it gets its own GUID.