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

Remove deprecated and unused ApiDocs command from setup.py #1925

Merged
merged 2 commits into from
Aug 1, 2023

Conversation

Kami
Copy link
Member

@Kami Kami commented Aug 1, 2023

This pull request removes deprecated and unused "apidocs" command from setup.py (apidocs are being generated using sphinx for a long time already).

This way it also removes dependency on distutils.core.Command which is not available anymore in Python 3.12.

distutils.core.Command has also been deprecated and removed in Python 3.12.
@Kami
Copy link
Member Author

Kami commented Aug 1, 2023

Keep in mind that this is just the first step. To get dist steps to pass under Python 3.12, we will need to get rid of setup.py all together and replace it with packaging metadata in setup.cfg / pyproject.toml file.

@codecov-commenter
Copy link

Codecov Report

Merging #1925 (ae71ed4) into trunk (72d331c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            trunk    #1925   +/-   ##
=======================================
  Coverage   83.12%   83.12%           
=======================================
  Files         353      353           
  Lines       81366    81366           
  Branches     8598     8598           
=======================================
  Hits        67631    67631           
  Misses      10931    10931           
  Partials     2804     2804           

@Kami Kami merged commit 24a455f into apache:trunk Aug 1, 2023
17 checks passed
@Kami Kami added this to the v3.8.0 milestone Aug 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants