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

check for existing roles/directives when registering new ones from an extension #1962

Closed
birkenfeld opened this issue Jul 21, 2015 · 0 comments
Milestone

Comments

@birkenfeld
Copy link
Member

From Peter Eastman on sphinx-users:
"""
It turns out Sphinx 1.3 introduced a new role called "numref", which is the same name the extension was using. It then silently ignored the extension and created instances of its own node type rather than the extension's type.

Glad to have that figured out. Could I make a feature request, though? If an extension defines a role or directive with the same name as a built in one, could it produce an error rather than silently ignoring it? That would have saved me hours of debugging.
"""

@tk0miya tk0miya added this to the 1.4 milestone Feb 14, 2016
t4ngo added a commit to t4ngo/sphinxcontrib-traceables that referenced this issue May 29, 2016
Sphinx added checking for registering nodes, roles, and directives which are already registered (see commit sphinx-doc/sphinx@cdf0f82 and sphinx-doc/sphinx#1962). That breaks the testing logic of sphinx-testing (see https://github.com/sphinx-doc/sphinx-testing) where subsequent tests cause reregistration warnings.

This change adds a local, slightly modified, version of Sphinx' built-in testing utilities.

Resolves #3.
t4ngo added a commit to t4ngo/sphinxcontrib-multilatex that referenced this issue Jun 6, 2016
Sphinx added checking for registering nodes, roles, and directives which
are already registered (see commit sphinx-doc/sphinx@cdf0f82 and
sphinx-doc/sphinx#1962). That breaks the testing logic of sphinx-testing
(see https://github.com/sphinx-doc/sphinx-testing) where subsequent
tests cause reregistration warnings.

This change adds a local, slightly modified, version of Sphinx' built-in
testing utilities.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants