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

Upgrade pydoc-markdown #2117

Merged
merged 60 commits into from Feb 4, 2022
Merged

Upgrade pydoc-markdown #2117

merged 60 commits into from Feb 4, 2022

Conversation

ZanSara
Copy link
Contributor

@ZanSara ZanSara commented Feb 2, 2022

pydoc-markdown was pinned to a 3.x version, while version 4.x are now stable since a while.

I propose to upgrade pydoc-markdown to make it compatible with Black: in this way, the same bot can apply code and documentation changes in a single commit and reduce the overall commit noise.

In addition, the latest pydoc-markdown seems to introduce some neat improvements, like permalinks to every method (not present before)

@PiffPaffM
Copy link
Contributor

@ZanSara I tested the new pydoc-markdown version some month ago. I remember that there were some issues with generating arguments. They were not rendered properly. Let's check how they are rendered on haystack.deepset.ai.

Copy link
Member

@julian-risch julian-risch left a comment

Choose a reason for hiding this comment

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

Rather than unpinning the version, I'd suggest to pin the latest major release if it renders correctly.

.github/workflows/update_docsstrings_tutorials.yml Outdated Show resolved Hide resolved
@ZanSara
Copy link
Contributor Author

ZanSara commented Feb 3, 2022

You're right @PiffPaffM, I've made a test deployment from the branch and indeed there are rendering issues: https://haystack-website-c7z7lp2nb-deepset-overnice.vercel.app/reference/primitives I'll try to fix it and keep testing there.

@ZanSara
Copy link
Contributor Author

ZanSara commented Feb 3, 2022

@PiffPaffM I think I managed to get pydoc-markdown to work properly. Have a look at the test deployment and let me know if you spot any issues 🙂

https://haystack-website-6u1fqzxin-deepset-overnice.vercel.app/reference/crawler

Copy link
Contributor

@PiffPaffM PiffPaffM left a comment

Choose a reason for hiding this comment

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

The rendered content looks good! Thanks for updating it.

@ZanSara ZanSara merged commit 957e78e into master Feb 4, 2022
@ZanSara ZanSara deleted the upgrade-pydoc-markdown branch February 4, 2022 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants