👌 Change non-fatal directive parsing errors to warnings

[chrisjsewell]

For non-fatal errors, such as;
faulty options syntax, unknown option keys, and invalid option values,
a warning is raised, but the directive is still run (without the erroneous options).
The warning is given the myst.directive_parse type, which can be suppressed.

[codecov]

Codecov Report

Base: 89.84% // Head: 90.17% // Increases project coverage by +0.33% 🎉

Coverage data is based on head (54b6ed2) compared to base (aa3f04d).
Patch coverage: 94.64% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #682      +/-   ##
==========================================
+ Coverage   89.84%   90.17%   +0.33%     
==========================================
  Files          23       23              
  Lines        2560     2586      +26     
==========================================
+ Hits         2300     2332      +32     
+ Misses        260      254       -6     

Flag	Coverage Δ
pytests	90.17% <94.64%> (+0.33%)	⬆️

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

Impacted Files	Coverage Δ
myst_parser/mocking.py	87.45% <75.00%> (-0.30%)	⬇️
myst_parser/parsers/directives.py	96.36% <95.45%> (+8.58%)	⬆️
myst_parser/mdit_to_docutils/base.py	93.03% <100.00%> (+0.02%)	⬆️
myst_parser/warnings_.py	96.00% <100.00%> (+0.08%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[rowanc1]

Very much want to see this in the JS side as well, it is a bit too strict.

[rowanc1]

I only looked at the tests, but this seems like a great addition to me 👍. I will follow suit in the JS side soon!

[chrisjsewell]

Very much want to see this in the JS side as well, it is a bit too strict.

yep exactly, I know we talked about this

Either in this PR or a follow, was going to add changing unknown directives and roles to a warning, with a certain warning type

[rowanc1]

Awesome, can you tag me so I can make the JS one the same!

[choldgraf]

was going to add changing unknown directives and roles to a warning, with a certain warning type

+1 from me