rewrite Path to address multiple issues

magmax / python-inquirer

A collection of common interactive command line user interfaces, based on Inquirer.js (https://github.com/SBoudrias/Inquirer.js/)

MIT License

1.01k stars 97 forks source link

rewrite Path to address multiple issues #587

Closed Cube707 closed 3 months ago

Cube707 commented 3 months ago

This PR aims to close #585 and close #586 by reconsidering bigger parts of the Path question-type. Mainly the validation function needs a rewrite.

Cube707 commented 3 months ago

@aaronhendry this version should address both issues, would you care to test it and give your opinion?

install via:

pip install -U "git+https://github.com/magmax/python-inquirer@fix/path"

aaronhendry commented 3 months ago

This fixes issue #585, which was the issue surrounding valid paths being rejected. It hasn't really fixed issue #586, but if you've decided to remove normalisation entirely from the Path query, then this issue isn't really relevant anymore.