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

Hook: Added hook "TTT2PlayDeathScream" for deathscreams #1668

Merged
merged 10 commits into from
Jan 3, 2025

Conversation

NickCloudAT
Copy link
Contributor

This hook allows to change the sound of the deathscream or cancel it outright.

Maybe I'm mistaken but I thought the table approach would be the most appropriate one here. Please correct me if not.

Changelog will follow when this is done.

@Aeconomy7
Copy link

Woohoo, awesome to see this making it in!

  • Sc00by

@NickCloudAT
Copy link
Contributor Author

(any feedback appreciated)

Changed isSilent to be a extra param in the hook itself. It will only be used by the GM hook if no other hook returns something.
@NickCloudAT
Copy link
Contributor Author

Latest commit changed to how isSilent should be handled instead, as per suggestion.

@NickCloudAT NickCloudAT marked this pull request as ready for review November 2, 2024 08:08
@Histalek Histalek added the type/feature New functionality label Nov 27, 2024
@NickCloudAT
Copy link
Contributor Author

Anything else for this to get merged? Just so it does not stale :P

@TimGoll
Copy link
Member

TimGoll commented Jan 3, 2025

Anything else for this to get merged? Just so it does not stale :P

image

@NickCloudAT
Copy link
Contributor Author

Anything else for this to get merged? Just so it does not stale :P

image

dangit, ofc I missed that. Should be resolved now.

@TimGoll
Copy link
Member

TimGoll commented Jan 3, 2025

dangit, ofc I missed that. Should be resolved now.

Style is still complaining :D

@NickCloudAT
Copy link
Contributor Author

dangit, ofc I missed that. Should be resolved now.

Style is still complaining :D

I might just be retarded.. But I actually don't get why it's complaining o.O?

@NickCloudAT
Copy link
Contributor Author

Ok.. ffs.. Took me 10 years now to know how the freak this style check works.. Well, better now than never.

@TimGoll
Copy link
Member

TimGoll commented Jan 3, 2025

Ok.. ffs.. Took me 10 years now to know how the freak this style check works.. Well, better now than never.

FYI, if you install stylua in VSCode it autodects the config file in the repo and does everything for you everytime you save the file

@TimGoll TimGoll merged commit 880ac92 into TTT-2:master Jan 3, 2025
4 checks passed
@NickCloudAT
Copy link
Contributor Author

Ok.. ffs.. Took me 10 years now to know how the freak this style check works.. Well, better now than never.

FYI, if you install stylua in VSCode it autodects the config file in the repo and does everything for you everytime you save the file

Ohhhhhhh wait actually? Damn, will do that rn.

@TimGoll
Copy link
Member

TimGoll commented Jan 3, 2025

Ohhhhhhh wait actually? Damn, will do that rn.

Yes! I wasn't a fan of it initially, but Histalek was right, it is really nice. If you have any issues with it, ask him, he knows how to fix it :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/feature New functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants