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

Heat damage testing #3

Closed
wants to merge 3 commits into from
Closed

Heat damage testing #3

wants to merge 3 commits into from

Conversation

Tostov
Copy link
Owner

@Tostov Tostov commented Sep 7, 2024

Additions

Describe new functionality added by your code, e.g.

  • Tribal smoke bombs
  • New tribal smoke bomb sprite
  • Tribal smoke bomb recipes at smithing bench and crafting spot using prometheum

Changes

Describe adjustments to existing features made in this merge, e.g.

  • Increased regular smoke bomb radius

References

Links to the associated issues or other related pull requests, e.g.

  • Closes #[ISSUE_NUMBER]
  • Contributes towards #[ISSUE_NUMBER]

Reasoning

Why did you choose to implement things this way, e.g.

  • Tribals need ways to close distance with pirate raiders
  • Smoke bombs allow this while enhancing combat micro
  • Thematically appropriate as we already allow tribal prometheum handling
  • Easy to implement
  • Buffed regular smoke grenades as they are rarely utilized and to justify additional investment

Alternatives

Describe alternative implementations you have considered, e.g.

  • Tribal catapult that launches melee animals into siege camps:
    • Additional use for animals
    • Anachronistic
    • Breaks realism theme

Testing

Check tests you have performed:

  • Compiles without warnings
  • Game runs without errors
  • (For compatibility patches) ...with and without patched mod loaded
  • Playtested a colony (specify how long)

@Tostov Tostov closed this Sep 7, 2024
@Tostov Tostov deleted the Heat-damage-testing branch September 7, 2024 08:32
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.

1 participant