Closed Brilliand closed 2 years ago
Great idea to set a better default value but two questions:
TradeConstants
necessary?defaultSelection
is supposed to be if it's not maxSelection
- maybe just breaking it down to one or two more named variables would make the code a bit easier to read?Thanks for taking the time to clarify the code a bit. 👍
I don't think this is the right place to change what the trade partners buy, but I see why swapping the order of food/water could be convenient. Would you also change that to a swap instead of delete?
This addresses issue #167