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

Maak mkdocs-material-insiders een productie requirement #566

Conversation

robbertbos
Copy link
Member

Beschrijf jouw aanpassingen

Deze PR lost het probleem op dat als je geen toegang hebt tot mkdocs-material-insiders, je het algoritmekader lokaal niet kan runnen. Zie #557.

Bij welk issue hoort deze pull-request?

#557

Checklist before requesting a review

  • Ik heb de contributing guidelines van deze repository gelezen en gevolgd.
  • Ik heb mijn aanpassingen gecheckt op spelfouten.
  • Als ik gebruik heb gemaakt van links, dan heb ik gecheckt of deze werken.
  • Ik heb gebruik gemaakt van de templates en formats van het algoritmekader.

Copy link

github-actions bot commented Jan 11, 2025

PR Preview Action v1.6.0
Preview removed because the pull request was closed.
2025-01-14 09:06 UTC

@robbertbos
Copy link
Member Author

@floort als het goed is, zou je deze branch lokaal moeten kunnen draaien. De mkdocs-material-insiders is nu een productie requirement. Door alleen requirements.txt te installeren krijg je de publiek toegankelijke versie van mkdocs-material en deze zou moeten werken. Het enige is dat de breadcrumbs dan niet werken, maar wat mij betreft is dat niet een probleem.

@ruthkoole kun jij ook naar deze voorgestelde oplossing kijken?

@floort
Copy link
Contributor

floort commented Jan 13, 2025

Dat lijkt goed te werken.

Copy link
Collaborator

@ruthkoole ruthkoole left a comment

Choose a reason for hiding this comment

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

LGTM

@ruthkoole ruthkoole force-pushed the 557-algoritmekader-niet-meer-te-testendeployen-zonder-licentie-van-15-dollar-per-maand branch from 340469b to b26aca5 Compare January 14, 2025 09:04
@ruthkoole ruthkoole merged commit 776d395 into main Jan 14, 2025
1 check passed
@ruthkoole ruthkoole deleted the 557-algoritmekader-niet-meer-te-testendeployen-zonder-licentie-van-15-dollar-per-maand branch January 14, 2025 09:06
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.

Algoritmekader niet meer te testen/deployen zonder licentie van 15 dollar per maand
3 participants