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

Remove old examples.tutorial schema #2107

Merged
merged 5 commits into from
Sep 13, 2024
Merged

Remove old examples.tutorial schema #2107

merged 5 commits into from
Sep 13, 2024

Conversation

evansd
Copy link
Contributor

@evansd evansd commented Sep 13, 2024

This isn't used and wasn't referenced anywhere in the tutorial, but it got included in the auto-generated docs and showed up in search results where it caused confusion.

We also fix a broken link which MkDocs was warning about and correct some not-actually-broken links to keep MkDocs happy and avoid warning fatigue.

I spotted the MkDocs warning about this, but I don't remember seeing
that before so possibly this is a consequence of upgrading MkDocs.
Copy link

cloudflare-workers-and-pages bot commented Sep 13, 2024

Deploying databuilder-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: c3151f1
Status: ✅  Deploy successful!
Preview URL: https://8334152b.databuilder.pages.dev
Branch Preview URL: https://evansd-remove-old-schema.databuilder.pages.dev

View logs

These weren't actually broken links, they were just specified in a way
where MkDocs was unable to verify them. But it's good to remove the
warning noise so we don't miss genuine problems.
This isn't used and wasn't referenced anywhere in the tutorial, but it
got included in the auto-generated docs and showed up in search results
where it caused confusion.
@evansd evansd merged commit 295cbae into main Sep 13, 2024
8 checks passed
@evansd evansd deleted the evansd/remove-old-schema branch September 13, 2024 15:38
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