sphinx-doc / sphinxcontrib-django

This is a sphinx extension which improves the documentation of Django apps.
https://pypi.org/project/sphinxcontrib-django/
Apache License 2.0
43 stars 25 forks source link

Fix: Abstract models should not show 'database table' string in rendered documentation #40

Closed insspb closed 1 year ago

insspb commented 1 year ago

This fixes issue, raised by me in #39

It also rename database model related options, and add few other.

fixes #39

insspb commented 1 year ago

@timoludwig, please take a look. I am plan to add few more pull request on things, that bothers me.

But as all of them related to model generation, I do not want to add all at once, to exclude merge conflicts.

codecov-commenter commented 1 year ago

Codecov Report

Merging #40 (89edbac) into main (955853e) will not change coverage. The diff coverage is 100.00%.

:exclamation: Current head 89edbac differs from pull request most recent head 2f0a76b. Consider uploading reports for the commit 2f0a76b to get more accurate results

@@            Coverage Diff            @@
##              main       #40   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines          308       314    +6     
=========================================
+ Hits           308       314    +6     
Impacted Files Coverage Δ
sphinxcontrib_django/docstrings/__init__.py 100.00% <100.00%> (ø)
sphinxcontrib_django/docstrings/classes.py 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

insspb commented 1 year ago

@timoludwig I removed django_tables_names_abstract_name option. Second PR (#42) fixed too.