pytest-sphinx icon indicating copy to clipboard operation
pytest-sphinx copied to clipboard

(WIP) Myst parser: `.md` + `{eval-rst}` support

Open tony opened this issue 3 years ago • 7 comments

re: #30

Naïve eval-rst directive support.

  • Support .md
  • Adds testdata/using_the_shapereader.md
  • Tests using_the_shapereader.md in test_cartopy
  • Support for {eval-rst} and tests
  • _DIRECTIVE_RE improvements - Use verbose, named expression for readability (cherry-picked and rebased in via #33, #34)

tony avatar Jul 12 '22 01:07 tony

@thisch I will ping you when this is ready to look at

Have not hand tested yet. I will build a very robust testsuite for it

tony avatar Jul 12 '22 01:07 tony

Awesome! Thx a lot.

twmr avatar Jul 12 '22 05:07 twmr

@thisch feel free to let tests run

tony avatar Jul 12 '22 11:07 tony

@tony thx a lot for all your PRs. I've just enabled the CI pipeline.

twmr avatar Jul 12 '22 20:07 twmr

@thisch Thank you!

  • Rebased after #33 merged in e3cf48b, #32 in 7eb7c6a
  • Created #34 for named groups
  • Removed typing.Literal for python 3.7 support (CI run). This is available for typing-extensions but its probably best to stay lean and keep it to standard library for now

Current PR status: I need to test this by hand, but open to any early feedback

tony avatar Jul 12 '22 20:07 tony

@thisch I suppose if you want you could cut a 0.4.1 release with the latest regex changes (via #33, #34)

I assume this PR, if merged (I still need to test it by hand and QA it more intensively) will be a bigger version bump, e.g. 0.5.0/0.6.0

tony avatar Jul 12 '22 21:07 tony

@thisch I may take a fresh stab at this non-docutils PR now that I am familiarized with doctest 👍

This approach, if pulled it off, would be easier to merge, preserve existing features, and have 0 extra dependencies.

tony avatar Sep 07 '22 01:09 tony