-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Proper implementation of pledge to protect #5165
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
Conversation
With all this sets out to do I don't see how it could be much shorter. |
I mean we could dispense with |
I'll implement this.
This is a quest, not related to pledge to protect. I'll have a look at city-state quests sometime in the future. Meanwhile could you add this to #4602 ?
Already implemented in #5133 |
I'm okay with the current implementation. Are the turns game-speed-agnostic? |
In the civ V source there's no correction for game speed, however there's a comment about adding it sometime in the future. I've preserved this, but we could just as easily put in multipliers. Do we want to? |
Nah, I think this is fine |
Protect gives resting point 10 for influence
sorry for late response, does this implement this?
|
No, those will be implemented in a future PR dealing with quests. |
This PR implements proper diplomacy penalties for aggressions against protected city-states.
HOWEVER, as you can see this is very complicated, includes a bunch of auxiliary timers and flags, and could alternatively be implemented almost as well with the existingActually this is OKrevertToZero
function. Do we want the elegant and simple way or the technically accurate way?