Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolve invalid escape sequence warnings #507

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

daugihao
Copy link

Simple change to prefix any regular expression matching strings with r to ensure they are used as raw strings and do not throw an invalid escape sequence warning:
image

@PaulSvensson
Copy link

This fix is needed and long due.
How can I help make it happen?

@daugihao
Copy link
Author

daugihao commented Jan 4, 2023

This fix is needed and long due. How can I help make it happen?

Thanks for supporting this PR, @PaulSvensson! I've not contributed to docopt before, so I'm not sure how to help expedite this change. Any advice welcome

@daugihao
Copy link
Author

daugihao commented Jan 4, 2023

The last release of this package was in 2014, so it looks like it's no longer maintained at all

@amotl
Copy link

amotl commented Apr 9, 2023

Dear @keleshev, @mboersma, and @graingert,

does one of you have the powers to integrate this patch and run a new release? It would be so sweet!

With kind regards,
Andreas.

@sodul
Copy link

sodul commented Nov 22, 2023

AFAIK these were fixed in 2018 in #435 but it was never merged or released.

@amotl
Copy link

amotl commented Mar 29, 2024

Hi again. Shall we do something about this package for those who still need it, like maintenance, hereby asking for permission? @bob-schumaker also submitted a similar patch at GH-523, right?

Also pinging the authors and most recent committers, @keleshev, @yb66, @mboersma, @graingert about it. Thank you very much in advance. 🙏

@amotl amotl mentioned this pull request Mar 29, 2024
@amotl
Copy link

amotl commented Mar 29, 2024

See @Sylvain303's answer at #519 (comment):

There is docopt-ng now, maintained by @jazzband. Cheers, @itdaniher, @johnscillieri, @nyurik, @hugovk, @jezdez, @NickCrews, @sturmianseq, and @h4l. 💙

@bob-schumaker
Copy link

bob-schumaker commented Mar 29, 2024 via email

@amotl
Copy link

amotl commented Mar 30, 2024

Maybe someone has the powers to update the README with a prominent admonition at the top to refer to docopt-ng, and after that, press the "Archive Repository" button?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants