yes, this will dumb of me to call it a String.  The current code just checks that params has a key named ```dryrun```, not even it's value,  like this ```import_puppetclasses?dryrun```, so it's kind of a boolean, but not really.  There is no sense in ```import_puppetclasses?dryrun=false```, since I would expect the user to call it without ```dryrun``` as a param.