Skip to content

Medical - Add configurable armor pain system with threshold - #266

Closed
fedoraman737 wants to merge 65 commits into
acemod:masterfrom
fedoraman737:medical/armor-pain-rewrite
Closed

Medical - Add configurable armor pain system with threshold#266
fedoraman737 wants to merge 65 commits into
acemod:masterfrom
fedoraman737:medical/armor-pain-rewrite

Conversation

@fedoraman737

@fedoraman737 fedoraman737 commented Sep 14, 2025

Copy link
Copy Markdown
Contributor

When merged this pull request will:

Requires:

Kexanone added 30 commits June 10, 2025 17:15

@veteran29 veteran29 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@veteran29
veteran29 requested a review from Kexanone September 14, 2025 15:28
@Kexanone Kexanone added kind/feature Release Notes: **ADDED:** status/blocked Waiting for dependency to be merged first labels Sep 14, 2025
@Kexanone Kexanone added this to the 1.4.0 milestone Sep 14, 2025
@Kexanone Kexanone modified the milestones: 1.4.0, Backlog Oct 20, 2025
@Kexanone

Kexanone commented Oct 23, 2025

Copy link
Copy Markdown
Member

Thanks for the PR, make sure to add yourself to AUTHORS.txt

@Kexanone

Kexanone commented Oct 26, 2025

Copy link
Copy Markdown
Member

Maybe we should limit the pain armor hits can cause. For instance, if the current pain is beyond moderate, it shouldn't pass more damage.

@Kexanone
Kexanone deleted the branch acemod:master October 30, 2025 21:03
@Kexanone Kexanone closed this Oct 30, 2025
@Kexanone Kexanone reopened this Nov 9, 2025
@Kexanone
Kexanone changed the base branch from medical/rewrite to master November 9, 2025 00:42
@Kexanone

Kexanone commented Nov 9, 2025

Copy link
Copy Markdown
Member

Closed in favor of #303

@Kexanone Kexanone closed this Nov 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature Release Notes: **ADDED:** status/blocked Waiting for dependency to be merged first

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants