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

Changes to Organ-Stabilizing property #4117

Closed
wants to merge 16 commits into from

Conversation

BeagleGaming1
Copy link
Contributor

@BeagleGaming1 BeagleGaming1 commented Aug 6, 2023

About the pull request

Changes Organ-Stabilizing from an on/off flag to being dependent on the organs damage

Explain why it's good for the game

Makes organ damage matter more if you get injured enough

Changelog

🆑
balance: Changes to research Organ-stabilizing property
/:cl:

@github-actions github-actions bot added the Balance You need to be a professional veteran game maintainer to comprehend what is being done here. label Aug 6, 2023
Copy link
Member

@morrowwolf morrowwolf left a comment

Choose a reason for hiding this comment

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

I think having it be a reduction in effect rather than complete turn off would be better. It would be less effective as the organ damage became worse.

Peri should be a way to limp away as someone with organ damage. I'd like a negative effect of some sort associated.

@morrowwolf morrowwolf marked this pull request as draft August 6, 2023 10:04
@BeagleGaming1 BeagleGaming1 marked this pull request as ready for review August 6, 2023 20:54
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Aug 11, 2023
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Aug 12, 2023
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@github-actions
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Aug 20, 2023
@morrowwolf
Copy link
Member

Gonna come back to this in the future

@morrowwolf morrowwolf closed this Sep 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Stale beg a maintainer to review your PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants