-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
mkdocs-git-committers-plugin-2 has an invalid wheel [was: Changes in MkDocs v1.4.0 causing deploy error] #21
Comments
Thanks for the detailed report @josh-wong ! From the error message, it looks like two versions of the plugin were installed: I will double check in parallel if there is an incompatibility with mkdocs 1.4.0. Let me know. |
Thanks for the additional troubleshooting pointers, @ojacques! I thought that reference to - run: pip install \
git+https://${GH_TOKEN}@github.com/squidfunk/mkdocs-material-insiders.git \
mkdocs-video \
mkdocs-git-revision-date-localized-plugin \
python-slugify \
mkdocs-git-committers-plugin-2 \
mkdocs-rss-plugin \ Example source: josh-wong/bitcoin-cash-node-on-raspberry-pi/.github/workflows/ci.yml Also, just to clarify since my
Sorry, I should have specified that I'm deploying the site using GitHub Pages, which is where I'm encountering the error. When serving the site locally, the site builds fine without encountering the error (although contributors aren't show on the locally built site, but I don't clearly remember if the contributors were displaying before). Locally, I can confirm that I've installed the latest version of I'll also do a little poking around to see what could be causing the issue and share any notes. |
@josh-wong, I found out that specifying a more restricted version range does work for - run: pip install \
git+https://${GH_TOKEN}@github.com/squidfunk/mkdocs-material-insiders.git \
mkdocs-video \
mkdocs-git-revision-date-localized-plugin \
python-slugify \
mkdocs-git-committers-plugin-2>=1.1.1 \
mkdocs-rss-plugin \ Yes, your build does not seem to work entirely. Something with the RSS plugin. You may want to have a look around [Edit] for RSS plugin, see this thread |
So to be clear, it seems like this is nothing about "Changes in MkDocs v1.4.0". |
True. I changed the title. |
Thanks a bunch, @ojacques! I've disabled And apologies for the busy work. I hope you can understand that, from my side, it looked like the changes to plugins in MkDocs v1.4.0 caused For future reference, I'll make a note to myself about how changes to MkDocs may cause conflict with other plugins just to avoid banging on the wrong door😅 |
Again,.. it's not about changes to MkDocs. |
Thanks for the explanation, @oprypin!
I understand that, but please see it from my angle (as a non-developer) that, after MkDocs v1.4.0 got released with changes to how plugins are implemented, a plugin breaks and I have to assume which one is broken simply by judging the contents of the error message. Based on that error mesage, I assumed that I know it's hard to coordinate all plugin developers to update or follow standard conventions, so I appreciate your work in testing plugins when releasing new versions of MkDocs🙌 |
No worries @josh-wong - thanks again for a great report. Feel free to do one again if you suspect something with this plugin! |
Issue
I attempted to deploy a docs site built on Material for MkDocs v8.5.5 with
mkdocs-git-committers-plugin-2
installed. However, the following error occurs:Screenshot
Troubleshooting
I've confirmed that the error does not occur when deploying docs sites built on Material for MkDocs v8.5.4, which is based on a version of MkDocs prior to v1.4.0.
As mentioned in the section "Upgrades for plugin developers" of the release notes for MkDocs v1.4.0, I think the changes to how MkDocs v1.4.0 and later handle plugins are causing this error. However, I'm not experienced nor knowledgeable enough to even try to make the changes on my own🙇🏻♂️
The text was updated successfully, but these errors were encountered: