Skip to content

Medical - Add configurable armor pain system with threshold#266

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

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

Conversation

@fedoraman737
Copy link
Contributor

@fedoraman737 fedoraman737 commented Sep 14, 2025

When merged this pull request will:

Requires:

Kexanone added 30 commits June 10, 2025 17:15
Copy link
Member

@veteran29 veteran29 left a comment

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
Copy link
Member

Kexanone commented Oct 23, 2025

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

@Kexanone
Copy link
Member

Kexanone commented Oct 26, 2025

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
Copy link
Member

Kexanone commented Nov 9, 2025

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