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

Work around harvester unloading activity queue issues. #16253

Merged
merged 1 commit into from Mar 2, 2019

Conversation

Projects
None yet
2 participants
@pchote
Copy link
Member

pchote commented Mar 2, 2019

This PR works around the regressions introduced by #15856 - hopefully without introducing any new ones. Harvesters still doesn't support queueing (see #16193) so this is probably going to be fine.

Fixes #16192.
Fixes #16249.
Probably fixes the complained about but not officially reported "harvesters go idle in resource patches" issue, as explained in #15856.

@pchote pchote added this to the Next Release milestone Mar 2, 2019

@reaperrr
Copy link
Contributor

reaperrr left a comment

Definitely fixes #16249 and couldn't reproduce #16192 on this, either

@reaperrr reaperrr merged commit 666d2e7 into OpenRA:bleed Mar 2, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.