alces-software / adminware

A sandbox CLI for running commands remotely across nodes
1 stars 0 forks source link

Fix/no genders error #83

Closed DavidMarchant closed 5 years ago

DavidMarchant commented 5 years ago

Fixes #71 and #81 when merged

How do you feel about the use of regex to sanitize nodenames? William mentioned using click but I wasn't really sure what how I would go about doing that Did these in the same branch as they both dealt with the genders file - i know this isn't optimum, sorry

DavidMarchant commented 5 years ago

Ah right! sorry @WilliamMcCumstie, combination of two attempts there, fixed: https://github.com/alces-software/adminware/pull/83/commits/d861777ca0e73d2f113402ede2f675c042ef3ecd