Skip to content

docs: restructure sphinx documentation#484

Merged
chrismattmann merged 1 commit into
chrismattmann:masterfrom
afuetterer:443-docs
Jun 10, 2026
Merged

docs: restructure sphinx documentation#484
chrismattmann merged 1 commit into
chrismattmann:masterfrom
afuetterer:443-docs

Conversation

@afuetterer

@afuetterer afuetterer commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

This will be a longer PR to fix #443.

I will set it to draft for now.

Waiting for related PRs

Tasks

  • rebase
  • add "documentation" to project.urls in pyproject.toml
  • remove readme.rst
  • include "readme.md" in index.rst
  • update docs/source/conf.py
  • add more badges to readme chore: add more informational badges to readme #493
  • fix documentation gha workflow
  • add docs build to ci to test if a pr breaks the docs?
  • workflow call, call from pr as well
  • if pr, do not deploy...
  • clean up documentation.yml
  • fix docstrings in python modules
  • clean up conf.py
  • squash commits
  • configure the GitHub actions workflow as the source of your deployment (needs to be done by @chrismattmann in repo settings)

@afuetterer afuetterer marked this pull request as draft April 21, 2026 07:22
@coveralls

coveralls commented Apr 21, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 67.356% (+0.3%) from 67.086% — afuetterer:443-docs into chrismattmann:master

@afuetterer

Copy link
Copy Markdown
Contributor Author

For these changes to take effect, please configure the GitHub actions workflow as the source of your deployment. Otherwise you will still have only the readme on your page.

See instructions: https://docs.github.com/en/pages/getting-started-with-github-pages/configuring-a-publishing-source-for-your-github-pages-site#publishing-with-a-custom-github-actions-workflow

@chrismattmann chrismattmann self-assigned this Apr 26, 2026
@chrismattmann chrismattmann added this to the tika-next milestone Apr 26, 2026
@chrismattmann

Copy link
Copy Markdown
Owner

For these changes to take effect, please configure the GitHub actions workflow as the source of your deployment. Otherwise you will still have only the readme on your page.

See instructions: https://docs.github.com/en/pages/getting-started-with-github-pages/configuring-a-publishing-source-for-your-github-pages-site#publishing-with-a-custom-github-actions-workflow

I will take a look how to do this this week, thanks @afuetterer the pre-req PRs are now merged.

@afuetterer

afuetterer commented May 11, 2026

Copy link
Copy Markdown
Contributor Author

What do you think about the following suggestions?

  • Build the docs in ci (e.g. on every pr) to see if something breaks -> skip deploy step
  • Build and push/deploy on every commit - now decide on whether push to master or only on tagged versions

@afuetterer

Copy link
Copy Markdown
Contributor Author

For these changes to take effect, please configure the GitHub actions workflow as the source of your deployment. Otherwise you will still have only the readme on your page.
See instructions: https://docs.github.com/en/pages/getting-started-with-github-pages/configuring-a-publishing-source-for-your-github-pages-site#publishing-with-a-custom-github-actions-workflow

I will take a look how to do this this week, thanks @afuetterer the pre-req PRs are now merged.

Did you look at this setting?

@afuetterer

Copy link
Copy Markdown
Contributor Author

Build and push/deploy on every push to master?
OR just on tagged releases?

Both are fine, I think.

@chrismattmann

Copy link
Copy Markdown
Owner

on every push

@chrismattmann

Copy link
Copy Markdown
Owner

@afuetterer i changed to "deploy from a branch" in Github settings for this repo as you requested.

@afuetterer

Copy link
Copy Markdown
Contributor Author

Alright, lets see if this works. Lets merge this and have a look at the docs. Okay?

@afuetterer afuetterer marked this pull request as ready for review June 8, 2026 07:47
@chrismattmann chrismattmann merged commit 7603ea4 into chrismattmann:master Jun 10, 2026
4 of 8 checks passed
@chrismattmann

Copy link
Copy Markdown
Owner

Merged, thanks @afuetterer

@afuetterer afuetterer deleted the 443-docs branch June 10, 2026 18:46
@afuetterer

Copy link
Copy Markdown
Contributor Author

It worked. What do you think about the docs? Improvement?

@chrismattmann

Copy link
Copy Markdown
Owner

Yes I love it thank you @afuetterer ! https://chrismattmann.github.io/tika-python/

Thank you for all your contributions so far! do you know why the build isn't passing?

@afuetterer

Copy link
Copy Markdown
Contributor Author

Awesome. Maybe you add the page as website under "Edit repository details". You can select "Use your GitHub Pages website".
Otherwise its kind of hard to discover, right?

CI is red, because of HTTP requests getting 429 responses.

@chrismattmann

Copy link
Copy Markdown
Owner

I'll add the page in repo details sounds good. how do we fix the build?

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.

Question about documentation

3 participants