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

Cherrypick "Shoot Over Bodies" And Related PRs #479

Merged

Conversation

VMSolidus
Copy link
Member

Description

This is a manual cherry-pick of the following PRs:
space-wizards/space-station-14#27905
space-wizards/space-station-14#28072
space-wizards/space-station-14#28571

I REQUIRE these for my work in PR #11 , and cannot complete said PR without these cherry-picks. This set of PRs from Wizden adds a feature where entities can selectively opt-out of being shot at unless a player intentionally targets them, which I can use as a simple and elegant solution to one of the largest glaring issues for Segmented Entities. I could simply give Lamia segments the new RequireProjectileTargetComponent, which adds them to the system. Future segmented entities such as the hypothetical "Heretic Worm" may or may not use this feature, depending on their intended implementation.

@VMSolidus VMSolidus added Changes: C# Changes any cs files Changes: YML Changes any yml files Size: 3-Medium For medium issues/PRs Priority: 1-Urgent Must be resolved immediately Type: Port Brings something to here from another codebase labels Jun 19, 2024
@DangerRevolution DangerRevolution added Status: Needs Review Someone please review this Pull Request Mirror Mirrors a PR from another Repo. Automatically applied by mirror bot labels Jun 20, 2024
@DangerRevolution
Copy link
Contributor

Copy the bodies over from the previous PR's; so whoever comes looking doesn't have to dive down several links (and because I also don't want to dive down these links)

@VMSolidus
Copy link
Member Author

Copy the bodies over from the previous PR's; so whoever comes looking doesn't have to dive down several links (and because I also don't want to dive down these links)

Yea I just checked, I actually can't do that.

@VMSolidus VMSolidus merged commit 40d411b into Simple-Station:master Jun 23, 2024
13 checks passed
@DEATHB4DEFEAT DEATHB4DEFEAT removed the Pull Request Mirror Mirrors a PR from another Repo. Automatically applied by mirror bot label Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: YML Changes any yml files Priority: 1-Urgent Must be resolved immediately Size: 3-Medium For medium issues/PRs Status: Needs Review Someone please review this Type: Port Brings something to here from another codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants