notitg-mirin icon indicating copy to clipboard operation
notitg-mirin copied to clipboard

std helper function names are inconsistent with casing.

Open ChocoboGamer opened this issue 2 years ago • 1 comments

the functions provided by the template have fairly inconsistent capitalization.

ie.

get_plr
touch_mod
touch_all_mods
sprite
aft
setupJudgeProxy
backToSongWheel

all of these should be switched to either camelCase with no _ or something else. Same with func_ease as it's the only mod application function with an _ it should be one word, like setdefault or definemod.

ChocoboGamer avatar May 07 '22 00:05 ChocoboGamer

if we keep the old ones around depricated I would love to switch all of them! I think we agreed that camelCase was the target

XeroOl avatar Sep 27 '22 17:09 XeroOl

I would like to suggest that we make function names in the xero namespace PascalCase and variables in the xero namespace should use camelCase.

ChocoboGamer avatar May 25 '23 19:05 ChocoboGamer