macadmins / installapplications

A tool for dynamically using installapplication
Apache License 2.0
285 stars 62 forks source link

generatejson - cast retries and retrywait to ints #109

Closed rickheil closed 1 year ago

rickheil commented 1 year ago

By default, argparse is using str for the retries and retrywait. This change forces a cast to int so that we don't end up with strings being read into IAs itself (throws a TypeError when it hits the retries code path).

Ideally I'd have argparse validate this and just store it as an int, but with using nargs I can't find an elegant way to do that without a lot of refactoring and I don't think that's worth it.