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

Modernize doc generation #880

Merged
merged 3 commits into from Jan 16, 2024
Merged

Modernize doc generation #880

merged 3 commits into from Jan 16, 2024

Conversation

4383
Copy link
Member

@4383 4383 commented Jan 12, 2024

Related to #879

  1. Use tox to build docs:
  • Move files into doc/source
  • Create a new testenv related to docs.
  • Manage docs requirements into a dedicated file
  • Remove obsolete makefile and build artifacts
  1. Run docs generation on pull requests, too many docstring errors have been accepted, leading us to broken docs generation.

  2. Fix a couple of error related to docs.

  3. Setup the eventlet repo to host the doc on readthedocs rather than on github pages. Docs are automatically generated each time we will push changes (commits, tags), no needs to run doc generation manually and to push it to github-pages.

Copy link

codecov bot commented Jan 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (81d4359) 55% compared to head (5442a10) 55%.

Additional details and impacted files
@@          Coverage Diff          @@
##           master   #880   +/-   ##
=====================================
  Coverage      55%    55%           
=====================================
  Files          90     90           
  Lines        9855   9855           
  Branches     1827   1827           
=====================================
  Hits         5437   5437           
  Misses       4050   4050           
  Partials      368    368           
Flag Coverage Δ
ipv6 22% <ø> (ø)
py310asyncio 52% <ø> (ø)
py310epolls 52% <ø> (-1%) ⬇️
py310poll 52% <ø> (+<1%) ⬆️
py310selects 52% <ø> (+<1%) ⬆️
py311asyncio 52% <ø> (+<1%) ⬆️
py311epolls 52% <ø> (ø)
py312asyncio 49% <ø> (+<1%) ⬆️
py312epolls 50% <ø> (ø)
py37epolls 50% <ø> (ø)
py38epolls 52% <ø> (ø)
py38openssl 51% <ø> (-1%) ⬇️
py38poll 52% <ø> (ø)
py38selects 52% <ø> (-1%) ⬇️
py39dnspython1 51% <ø> (+<1%) ⬆️
py39epolls 52% <ø> (ø)
py39poll 52% <ø> (ø)
py39selects 52% <ø> (-1%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@4383 4383 force-pushed the refactor-docs-generating branch 9 times, most recently from b1d18fc to e5b90d1 Compare January 16, 2024 09:26
@4383 4383 changed the title [WIP] Modernize doc generation Modernize doc generation Jan 16, 2024
@4383 4383 requested a review from itamarst January 16, 2024 11:23
Copy link
Contributor

@itamarst itamarst left a comment

Choose a reason for hiding this comment

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

Beyond previously mentioned stuff, there's:

  • A broken link in the README, "open a new issue".
  • I can't actually see that ReadTheDocs publishing works since I don't know what the URL is (or maybe it's just not public at the obvious URL?)

@@ -0,0 +1,32 @@
# .readthedocs.yaml
Copy link
Contributor

Choose a reason for hiding this comment

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

We should link to the new URL for docs in the README.

Copy link
Member Author

@4383 4383 Jan 16, 2024

Choose a reason for hiding this comment

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

I'll do another patch to update the links (in the pyproject.toml file too) once the docs will be online on readthedocs.

@@ -0,0 +1,32 @@
# .readthedocs.yaml
# Read the Docs configuration file
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we have a way to update the old docs to point to the new URL? Otherwise Google will point people at old docs and they won't know new ones exist.

Copy link
Member Author

@4383 4383 Jan 16, 2024

Choose a reason for hiding this comment

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

readthedocs allow custom domains. We could simply redirect the eventlet.org eventlet.net domain to the new readthedocs hosted instance. Every users would be automatically redirected.

Also, if necessary, we could publish an empty html index on the github page containing the following javascript instruction to redirect to the new docs hosted on readthedocs:

window.location.href = ‘https://eventlet.readthedocs.io/’;

@4383
Copy link
Member Author

4383 commented Jan 16, 2024

Beyond previously mentioned stuff, there's:

* A broken link in the README, "open a new issue".

* I can't actually see that ReadTheDocs publishing works since I don't know what the URL is (or maybe it's just not public at the obvious URL?)

The docs is not yet online on readthedocs as the readthedocs config file is mandatory to allow building. The snake bites its tail. The url will be https://eventlet.readthedocs.io/ and http://eventlet.rtfd.io/

@4383
Copy link
Member Author

4383 commented Jan 16, 2024

* A broken link in the README, "open a new issue".

Fixed by 131651a

- Move files into doc/source
- Create a new testenv related to docs.
- Manage docs requirements into a dedicated file
- Remove obsolete makefile and build artifacts
Also:
- fix path in the existing script
- adapt gitignore rule to track github actions
@4383 4383 merged commit b9c7216 into master Jan 16, 2024
28 checks passed
@4383 4383 deleted the refactor-docs-generating branch January 16, 2024 14:43
@4383 4383 mentioned this pull request Jan 18, 2024
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.

None yet

2 participants