Skip to content

Hotfix - Tweak game balance#95

Merged
Tapawingo merged 7 commits intomainfrom
hotfix/tweak-game-balance
Dec 14, 2025
Merged

Hotfix - Tweak game balance#95
Tapawingo merged 7 commits intomainfrom
hotfix/tweak-game-balance

Conversation

@Tapawingo
Copy link
Contributor

@Tapawingo Tapawingo commented Dec 4, 2025

Note

Depends On #94

When merged this pull request will:

  • Tweak unit stats, ammunition damage, scenario and terrain effects on LOS and movement
  • Tweak morale decay speed
  • Fix unit persistence to only persist forward
  • Fix issue where dead units where still in combat after death

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code.
  • I have tested my code.
  • I have documented my code in accordance with gdscript_documentation_comments.
  • I have requested a review in discord.
  • I have moved the related issues to the review section in the SCRUM Board.

@GianRathgeb
Copy link
Contributor

Are the sounds already in this commit? because when I load the branch from git, the sounds are already included in here. So I have the same issues as on that other branch. Can you please confirm?

@Tapawingo
Copy link
Contributor Author

Are the sounds already in this commit? because when I load the branch from git, the sounds are already included in here. So I have the same issues as on that other branch. Can you please confirm?

they are, some of the bugfixes in the sounds PR was required for the cahnges in this one. Hence the dependency on #94 before merge

Copy link
Contributor

@tobelobb tobelobb left a comment

Choose a reason for hiding this comment

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

looks good, found no isuues

@Tapawingo Tapawingo merged commit cad0a06 into main Dec 14, 2025
3 checks passed
@Tapawingo Tapawingo deleted the hotfix/tweak-game-balance branch December 14, 2025 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants