search

lukasschwab / arxiv.py

Python wrapper for the arXiv API
MIT License
1.11k stars 123 forks source link

Improve pdoc rendering #68

Closed mhils closed 3 years ago

mhils commented 3 years ago

Hi there!

Thanks for the excellent project - it's a nice feeling to work with code that then again makes use of one's own project. 😄

I noticed that you have an interesting single-page configuration for pdoc which is slightly awkward with pdoc's folder structure expectations. I just pushed a new pdoc release which supports this a bit better, here are the accompanying changes for arxiv.py. 😃

lukasschwab commented 3 years ago

👋 Hi! Thanks for your work––pdoc is great.

Well-spotted. Maybe it's a mistake in how I designed this package, but pdoc's index wasn't super useful for this project––it just lists one module, arxiv, so I figured I'd pull that module's docs page out to replace the index. Not sure that this is typical, but glad to hear it's better supported now!

Reviewing...

mhils commented 3 years ago

Well-spotted. Maybe it's a mistake in how I designed this package, but pdoc's index wasn't super useful for this project––it just lists one module, arxiv, so I figured I'd pull that module's docs page out to replace the index. Not sure that this is typical, but glad to hear it's better supported now!

pdoc always generates a module index for consistency, but doesn't link to it if only one module is being documented. In your case it detected two modules - arxiv (arxiv/__init__.py) and arxiv.àrxiv (arxiv/arxiv.py) - which is why the link was there.

FWIW you could also simplify your package to a single file (mv ./arxiv/arxiv.py ./arxiv.py; rm -rf ./arxiv), but I didn't want to impose any structural changes to your work here. :)