Just some minor followup while reviewing PRs that got merged recently. @catloversg FYI
I think the only thing added in #1365 that was an issue was some unnecessary typechecking code in the travel functions, but the other changes are all in the same modified code.
Player.travel and Sleeve.travel moved to Person.travel (they were identical implementations)
Removed some unnecessary typechecking code in travel and goToLocation (parameters already strongly typed, requiring typechecking before calling the function)
In TravelModal, removed an as CityName assertion and some other unnecessary typecode.
github-actions[bot]
commented
1 week ago
Just some minor followup while reviewing PRs that got merged recently. @catloversg FYI
I think the only thing added in #1365 that was an issue was some unnecessary typechecking code in the travel functions, but the other changes are all in the same modified code.
as CityNameassertion and some other unnecessary typecode.