Skip to content

🐛 FIX: Front matter -> sphinx metadata regression#480

Merged
chrisjsewell merged 1 commit intomasterfrom
fix-front-matter-smartquotes
Dec 29, 2021
Merged

🐛 FIX: Front matter -> sphinx metadata regression#480
chrisjsewell merged 1 commit intomasterfrom
fix-front-matter-smartquotes

Conversation

@chrisjsewell
Copy link
Copy Markdown
Member

@chrisjsewell chrisjsewell commented Dec 29, 2021

After PR #477, I noted that the SmartQuote transform was being applied to the front-matter values,
before they were extracted to the document metadata (see also issue #479).
Therefore, we now add them as a literal to stop this happening.

After #477, I noted that the `SmartQuote` transform was being applied to  the front-matter values,
before they were extracted to the document metadata.
Therefore, we now add them as a `literal` to stop this happening.
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 29, 2021

Codecov Report

Merging #480 (100dc84) into master (2b3a931) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #480   +/-   ##
=======================================
  Coverage   90.02%   90.02%           
=======================================
  Files          16       16           
  Lines        2056     2056           
=======================================
  Hits         1851     1851           
  Misses        205      205           
Flag Coverage Δ
pytests 90.02% <100.00%> (ø)

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

Impacted Files Coverage Δ
myst_parser/docutils_renderer.py 92.13% <100.00%> (ø)

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 2b3a931...100dc84. Read the comment docs.

@chrisjsewell chrisjsewell merged commit 58f0540 into master Dec 29, 2021
@chrisjsewell chrisjsewell deleted the fix-front-matter-smartquotes branch December 29, 2021 20:18
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.

1 participant