Should treat tool specs as case insensitive

LPGhatguy / aftman

Aftman, the prodigal sequel to Foreman

MIT License

162 stars 19 forks source link

Should treat tool specs as case insensitive #41

Open JohnnyMorganz opened 1 year ago

JohnnyMorganz commented 1 year ago

Right now, aftman treats e.g. UpliftGames/wally@0.3.1 and upliftgames/wally@0.3.1 as separate tools and will install the latter again even if the former is already installed. They should instead be treated as the same tool.

I'm unsure however if there are any negative side effects for it being case insensitive

LPGhatguy commented 1 year ago

Yeah, this is definitely something to fix. We can use manually implement PartialEq and Hash to use lowercase variants of names while preserving the original case.