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

👌 IMPROVE: directives.yml (spelling & description) #23

Merged
merged 1 commit into from
Jun 26, 2021

Conversation

jedbrown
Copy link
Contributor

I believe this fixes @mbercx's issue in #14 (comment). Specifically, one can now run rst2myst convert -e sphinx_panels input.rst and get the desired

:::{dropdown} MyST $\pi$ markdown title
and content {eq}`important-equation`
:::

@codecov
Copy link

codecov bot commented Jun 26, 2021

Codecov Report

Merging #23 (a0e04fd) into main (9f43c3a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #23   +/-   ##
=======================================
  Coverage   84.29%   84.29%           
=======================================
  Files          10       10           
  Lines        1617     1617           
=======================================
  Hits         1363     1363           
  Misses        254      254           
Flag Coverage Δ
pytests 84.29% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9f43c3a...a0e04fd. Read the comment docs.

@chrisjsewell chrisjsewell changed the title directives.yml: fix spelling and description of parse_all 👌 IMPROVE: directives.yml (spelling & description) Jun 26, 2021
@chrisjsewell chrisjsewell merged commit 7a6cbac into executablebooks:main Jun 26, 2021
@welcome
Copy link

welcome bot commented Jun 26, 2021

Congrats on your first merged pull request in this project! 🎉
congrats

Thank you for contributing, we are very proud of you! ❤️

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.

2 participants