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

Add ehrQL Pygments configuration #1372

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

StevenMaude
Copy link
Contributor

@StevenMaude StevenMaude commented Nov 7, 2023

Add the appropriate configuration to make ehrql code blocks display with syntax highlighting. These were introduced in opensafely-core/ehrql#1648.

Copy link

cloudflare-workers-and-pages bot commented Nov 7, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 289329c
Status: ✅  Deploy successful!
Preview URL: https://5e73f590.opensafely-docs.pages.dev
Branch Preview URL: https://steve-add-ehrql-pygments-con.opensafely-docs.pages.dev

View logs

We now use `ehrql` syntax labels for complete dataset definitions in the
ehrQL documentation: opensafely-core/ehrql#1648.

This documentation repository needs to know about this configuration
too. This duplication is a little bit irritating, but it is because our
multi-repository configuration doesn't have any kind of configuration
inheritance or reuse. See opensafely-core/ehrql#1126

A future improvement might be to use the Material for MkDocs projects
plugin once it's available in the free version which might also fix this
duplication: #1317.
@StevenMaude StevenMaude merged commit 57ef523 into main Nov 7, 2023
2 checks passed
@StevenMaude StevenMaude deleted the steve/add-ehrql-pygments-configuration branch November 7, 2023 14:41
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.

2 participants