Skip to content

Fix: Disable Myst deprecation warnings#704

Merged
bmunkholm merged 1 commit intomainfrom
bm/fix-deprecation-warnings
Feb 12, 2026
Merged

Fix: Disable Myst deprecation warnings#704
bmunkholm merged 1 commit intomainfrom
bm/fix-deprecation-warnings

Conversation

@bmunkholm
Copy link
Contributor

@bmunkholm bmunkholm commented Feb 12, 2026

About

Disable all the deprecation warnings as the build log is flooded by them. E.g. here.

/home/docs/checkouts/readthedocs.org/user_builds/cratedb-guide/envs/latest/lib/python3.14/site-packages/myst_nb/sphinx_.py:73:
RemovedInSphinx10Warning: 'myst_nb.sphinx_.Parser.env' is deprecated. Check CHANGES for Sphinx API modifications.

@coderabbitai

This comment was marked as spam.

@bmunkholm bmunkholm requested a review from amotl February 12, 2026 19:02
@bmunkholm bmunkholm enabled auto-merge (squash) February 12, 2026 19:03
@bmunkholm bmunkholm requested a review from kneth February 12, 2026 19:09
@bmunkholm bmunkholm self-assigned this Feb 12, 2026
Copy link
Member

@amotl amotl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for mitigating this. I don't have objections about the suppression of the warning, but would like to keep reporting and bookkeeping procedures sound.

Comment on lines +187 to +194
# Suppress myst-nb deprecation warnings until fixed upstream.
# https://github.com/executablebooks/MyST-NB/issues/645
warnings.filterwarnings(
"ignore",
message=r".*myst_nb\.sphinx_\.Parser\.env.*",
category=DeprecationWarning,
)

Copy link
Member

@amotl amotl Feb 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that is the wrong issue, and it is already closed: It reported about the sphinx.util.import_object symbol being deprecated.

May I ask you to please report your specific observation about the myst_nb.sphinx_.Parser.env symbol to the upstream project 1 and reference the new ticket here? 🙏

Footnotes

  1. Would it just be about asking for a new release, or does it need patches? 2

  2. Maybe https://github.com/executablebooks/MyST-NB/pull/706 fixed it already?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ups! Wrong copy/paste. Good catch 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a PR from Dec open that seems dormant: executablebooks/MyST-Parser#1079

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(updated with correct link directly in main)

@bmunkholm bmunkholm merged commit f39d082 into main Feb 12, 2026
9 checks passed
@bmunkholm bmunkholm deleted the bm/fix-deprecation-warnings branch February 12, 2026 19:30
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