-
Notifications
You must be signed in to change notification settings - Fork 176
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
Campaign now uses pickRandomCamouflage when initialized #5276
Conversation
Worked with all the start dates except the 3rd Jihad one but that one seems to be broken |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5276 +/- ##
============================================
- Coverage 10.47% 10.47% -0.01%
+ Complexity 6049 6048 -1
============================================
Files 959 959
Lines 135202 135203 +1
Branches 19674 19674
============================================
- Hits 14165 14163 -2
+ Misses 119686 119685 -1
- Partials 1351 1355 +4 ☔ View full report in Codecov by Sentry. |
What error does that give you? |
Campaign file could not be loaded. Gives same error when I switch to master too. |
Also realize that maybe I should limit the camos to "Standard Camouflage" instead of all camos |
log error |
Perfect, we’re still tracking down the last of those from work that was done in 50.01. I’ll get that fixed in an hour |
Let me know if I should switch to just "Standard Camouflage" instead of all. |
I think ‘all’ is perfectly fine. Gives a lot more choice and I don’t think anyone will mind if they get the same camo as a specific unit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking a look at this.
LGTM :)
Modified pickRandomCamo to have "Root" faction that selects from all faction icons and implemented it in Campaign.
closes #5114