Skip to content

Conversation

@nrichers
Copy link
Collaborator

@nrichers nrichers commented Oct 17, 2025

Pull Request Description

What and why?

This PR removes the following from the repo, as we now handle the generation of Quarto Markdown for the Python API reference on the fly at docs build time:

  • .github/workflows/quarto-docs.yaml - defunct workflow, now handled in the documentation repo at docs build time
  • docs/validmind.qmd, docs/validmind/ - defunct generated Quarto Markdown docs for the Python API

Relates to:

How to test

There is nothing to test here, as the magic happens in validmind/documentation#1001, so head on over there to try the PR preview. (There should be no difference in the output to before.)

What needs special review?

Dependencies, breaking changes, and deployment notes

Release notes

Checklist

  • What and why
  • Screenshots or videos (Frontend)
  • How to test
  • What needs special review
  • Dependencies, breaking changes, and deployment notes
  • Labels applied
  • PR linked to Shortcut
  • Unit tests added (Backend)
  • Tested locally
  • Documentation updated (if required)
  • Environment variable additions/changes documented (if required)

@nrichers nrichers added the internal Not to be externalized in the release notes label Oct 17, 2025
@nrichers nrichers marked this pull request as draft October 17, 2025 19:57
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@nrichers nrichers changed the title Remove Quarto Markdown output from repo & update readme Nrichers/sc 12743/fix quarto docs workflow issue Oct 18, 2025
@nrichers nrichers marked this pull request as ready for review October 18, 2025 00:37
@nrichers nrichers requested a review from nibalizer October 20, 2025 20:34
@nibalizer
Copy link
Contributor

@nrichers with the +6 -15k let's go

@nrichers nrichers merged commit a70f2e1 into main Oct 22, 2025
16 of 17 checks passed
@nrichers nrichers deleted the nrichers/sc-12743/fix-quarto-docs-workflow-issue branch October 22, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Not to be externalized in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants