Skip to content

Bump documentation website dependencies #169

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

Merged
merged 4 commits into from
May 16, 2023
Merged

Bump documentation website dependencies #169

merged 4 commits into from
May 16, 2023

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented May 16, 2023

The project's documentation is published in a MkDocs-based website:

https://arduino.github.io/arduino-fwuploader/dev/

One of the Python packages that is a transitive dependency of the website framework was recently updated (#168). This caused a regression in the way the website renders fenced code blocks.

Before that pymdown-extensions package update, the fenced code blocks were rendered as expected:

https://arduino.github.io/arduino-fwuploader/2.2/commands/arduino-fwuploader/#options

image

after the pymdown-extensions package update they were rendered incorrectly:

https://arduino.github.io/arduino-fwuploader/dev/commands/arduino-fwuploader/#options

image

I didn't identify the specific reason why this happened, but I did find that it was fixed by updating the mkdocs-material package dependency. While I was at it, I also updated the other MkDocs-related package dependencies of the website as they were all significantly outdated and experience (including this one) shows that they are best updated in parallel.


You can see a demonstration of how the website will look after the changes proposed here are applied here where I deployed it in my fork:

https://per1234.github.io/arduino-fwuploader/dev/commands/arduino-fwuploader/#options

@per1234 per1234 added topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project labels May 16, 2023
@per1234 per1234 requested a review from MatteoPologruto May 16, 2023 06:11
@per1234 per1234 self-assigned this May 16, 2023
Copy link
Contributor

@MatteoPologruto MatteoPologruto left a comment

Choose a reason for hiding this comment

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

Thanks Per!

@per1234 per1234 merged commit f7bfc92 into arduino:main May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants