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

py domain: Generate node_id for objects and modules in the right way #7236

Merged
merged 3 commits into from Mar 6, 2020

Conversation

tk0miya
Copy link
Member

@tk0miya tk0miya commented Mar 1, 2020

@tk0miya tk0miya added this to the 3.0.0 milestone Mar 1, 2020
@tk0miya tk0miya force-pushed the refactor_py_domain2 branch 2 times, most recently from b9a2546 to edf2238 Compare March 1, 2020 16:59
@@ -273,6 +273,7 @@ Additionally, the following filters are available
replaces the builtin Jinja `escape filter`_ that does html-escaping.

.. function:: underline(s, line='=')
:noindex:
Copy link
Member Author

Choose a reason for hiding this comment

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

I think this would be an another bug. I'll take a look carefully tomorrow.

@codecov
Copy link

codecov bot commented Mar 6, 2020

Codecov Report

Merging #7236 into 3.x will increase coverage by <.01%.
The diff coverage is 97.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##              3.x    #7236      +/-   ##
==========================================
+ Coverage   84.01%   84.01%   +<.01%     
==========================================
  Files         279      279              
  Lines       43087    43097      +10     
  Branches     6281     6282       +1     
==========================================
+ Hits        36198    36208      +10     
  Misses       5517     5517              
  Partials     1372     1372
Impacted Files Coverage Δ
tests/test_build_html.py 97.91% <ø> (ø) ⬆️
tests/test_intl.py 97.93% <ø> (ø) ⬆️
tests/test_domain_py.py 99.33% <100%> (ø) ⬆️
tests/test_environment.py 100% <100%> (ø) ⬆️
sphinx/domains/python.py 79.58% <95.55%> (+0.24%) ⬆️
sphinx/domains/cpp.py 80.6% <0%> (ø) ⬆️

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 c767f44...f13c546. Read the comment docs.

@tk0miya tk0miya merged commit f329553 into sphinx-doc:3.x Mar 6, 2020
@tk0miya tk0miya deleted the refactor_py_domain2 branch March 6, 2020 17:08
tk0miya added a commit to tk0miya/sphinx that referenced this pull request Mar 21, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant