More configuration options on `OsStrParser` (maybe making it a "PathParser`?)

aj-bagwell / clio

A rust library for parsing command line file name arguements

12 stars 7 forks source link

More configuration options on `OsStrParser` (maybe making it a "PathParser`?) #12

Closed epage closed 1 year ago

epage commented 1 year ago

In mapping click concepts to this library,

CachedInput, Input and Output line up with File
InputPath and OutputPath sort of line up with File with lazy = true
InputPath and OutputPath lines up with Path but none of Paths config options.

It seems like Path config options would belong as settings on OsStrParser so a user can pass in a custom value parser when the defaults expressed through the types don't quite work

aj-bagwell commented 1 year ago

I have finally got round to releasing v0.3 which adds a ClioPath for representing more generic paths.

As per this great suggestion I added extra validation options to OsStrParser for exists, is_file and is_dir.

I also added checking for read/write access to InputPath and OutputPath.