Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Final merge of dev branch into master, including alternate model loading #1268

Merged
merged 7 commits into from
Nov 1, 2017

Conversation

keveleigh
Copy link
Contributor

The Dev_Unity_2017.2.0 branch will be retired after this PR to prevent future work from being done in it.

StephenHodgson and others added 7 commits October 16, 2017 23:50
… AlwaysUseAlternateModel boolean to allow you to specify a default if the models aren't found/loaded or specify a model that should always be used.
Allowing default versus overriden controller models
@keveleigh keveleigh self-assigned this Oct 30, 2017
@StephenHodgson
Copy link
Contributor

yeah we should have caught that he submitted to the dev branch. Awe well.

@keveleigh
Copy link
Contributor Author

I caught it but figured it'd be easier to do it myself, since there had been some big changes to this script in master.

@StephenHodgson
Copy link
Contributor

Yeah I was tempted to delete that branch too, but wanted to check first if it was okay.

@keveleigh
Copy link
Contributor Author

After this, we should delete it.

@DerekMa-WP
Copy link

Sorry about that. Everything looks good to me.

@keveleigh
Copy link
Contributor Author

@MsDerekMa All good!

Copy link
Contributor

@StephenHodgson StephenHodgson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Worked well for me.

@keveleigh keveleigh merged commit 445c816 into microsoft:master Nov 1, 2017
@keveleigh keveleigh deleted the DevMerge branch November 1, 2017 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants