Closed docrostov closed 1 year ago
Skimmed through the commits and they looked okay to me. Agree with the changes you're leaving out as well.
Any reason to hold off until some of those other things in your first chunk of bullets get done? I prefer the strategy of lots of small changes instead of big batches, so I'd move to get this merged and do the others separately 🤷
sure, i guess that's fine. i ran it for a bit and fixed a few glitches that were screwing with it, i think it's fine to merge in now.
alright this is finally addressing #101 and bringing us up to matching with a ton of tiny TES changes i didn't notice in previous instances of comparing base tourguide to his repo. changes are well tracked in the billion commits in the branch, but in essence, this captures the big block of new things TES had versus the bundled/released tourguide version. here are some things missing that i would like to do before i merge this:
here are some things that were in TES' that i think i am not going to add but would like thoughts