RutgersGRID / CYOA_Game_Package

new repo for WebGL Build
0 stars 0 forks source link

Refactor step 3 -- rename scripts - "As a development group, we would like to have a common naming convention so we can all understand what the code is." #12

Open HornOfSaltff opened 3 months ago

HornOfSaltff commented 3 months ago

Rename scripts to help us understand what each script does These are a mix of clear and less clear, especially since there are a lot of ScriptNameTwos.

Unity suggests these be PascalCase, which is looks like was used for most (if not all) scripts already, so we're good there.

But some are less clear and should be spelled out so new devs can more easily guess what they do/what's in them.

CCSVtoSO - contextually, I'd assume this converts CSV files into Scriptable Objects, but I'm not sure (what's the first "C" for?).

TWPopulator only makes sense if I guess that TW is Trigger Warning. We should spell it out so it's clear.

When you rename these, you have to remember to rename the class in the script as well.