executablebooks / rst-to-myst

Convert ReStructuredText to MyST Markdown
https://rst-to-myst.readthedocs.io
MIT License
64 stars 11 forks source link

Does not work on Windows #35

Closed fmfreeze closed 1 year ago

fmfreeze commented 2 years ago

Describe the bug

When using on Windows10 with the command rst2myst convert *.rst it produces:

Usage: rst2myst convert [OPTIONS] [PATHS]...
Try 'rst2myst convert -h' for help.

Error: Invalid value for '[PATHS]...': Path '*.rst' does not exist.

I triple-checked the if I am in the right directory, where my .rst files are stored. Also tried quoting the path.

Does anybody experienced that issue? On Linux it worked as expected.

Reproduce the bug

  1. in a conda environment install with pip install rst-to-myst[sphinx]
  2. in terminal navigate to the folder where *.rst files are stored
  3. execute the command rst2myst convert *.rst

List your environment

No response

welcome[bot] commented 2 years ago

Thanks for opening your first issue here! Engagement like this is essential for open source projects! :hugs:
If you haven't done so already, check out EBP's Code of Conduct. Also, please try to follow the issue template as it helps other community members to contribute more effectively.
If your issue is a feature request, others may react to it, to raise its prominence (see Feature Voting).
Welcome to the EBP community! :tada:

Joinyy commented 2 years ago

@fmfreeze You can get it working when upgrading your click package to 8.1.3, this works for me on python 3.8 and 3.10 on windows.

slavakx commented 2 years ago

@fmfreeze You can get it working when upgrading your click package to 8.1.3, this works for me on python 3.8 and 3.10 on windows.

Yes it works when installed directly with pip. However, when installed via poetry I get the following error:

Because rst-to-myst (0.3.2) depends on click (>=7.1,<8.0)
and no versions of rst-to-myst match >0.3.2,<0.4.0, rst-to-myst (>=0.3.2,<0.4.0) requires click (>=7.1,<8.0)

It would be good to have this issue solved. I spent the whole day trying to understand what is wrong with the path

chrisjsewell commented 1 year ago

v0.3.3 (just released) now allows click v8