[ENHANCEMENT] Character Singing Overrides And No anim Notes

[Eviethecoder]

This pr allows for scripts to easily override the base sing anim function with a singOveride variable. it also adds a no animation note to the chart editor!

[lunacynne]

"no animation" note is useful !!

though for the anim override i feel like a better solution would be to add an optional noAnimation and suffix property to the notekind
that way the anim override can be determined with just the note kind script rather than the script also having to mess with and correctly keep track of the variable in the characters so it doesnt accidentally also disable sing anims for other note kinds...

[Eviethecoder]

"no animation" note is useful !!

though for the anim override i feel like a better solution would be to add an optional noAnimation and suffix property to the notekind that way the anim override can be determined with just the note kind script rather than the script also having to mess with and correctly keep track of the variable in the characters so it doesnt accidentally also disable sing anims for other note kinds...
YO THAT MAKES SENSE!!!! ill add that quickly here soon thanks for the idea

[lunacynne]

note kind params is an array of param objects for custom options

the best solution imo would be like
instead of them being part of the param structure they should be instead be just noAnimation:Bool and ?suffix:String variables in the note kind class
and then in the hit note function -> check note kind in data -> get note kind from registry if it exists -> check those 2 variables -> handle animation logic accordingly

[Eviethecoder]

ALRIGHT! now this system uses the existing notekind manager to grab the scripted notekind. if its null itl just do the default singing. but if its not itl check for a no anim variable that must be set in the script. as well as if a suffix has been set otherwise it just plays default sing anims.

[AbnormalPoof]

Please change the base branch to develop.

Side note: I do not recommend using the default branch main for pull requests since it's considered an anti-pattern. Just something to keep in mind in the future.

[Eviethecoder]

Please change the base branch to develop.

Side note: I do not recommend using the default branch main for pull requests since it's considered an anti-pattern. Just something to keep in mind in the future.

Alrighty! It has been changed!! Thank you for letting me know for the future

[AbnormalPoof]

A lot of the comments here are messy and need cleanup. I also noticed some weird changes to the code.

[Eviethecoder]

A lot of the comments here are messy and need cleanup. I also noticed some weird changes to the code.

i believe i did all the needed edits here! if any more pop up ill be sure to fix!

[Eviethecoder]

im so surprised this is a honor thanks yall