Skip to content

Re-connect OAS-specific md2html tests - #5569

Open
handrews wants to merge 3 commits into
OAI:mainfrom
handrews:cleanup/shared-md2html-fixtures
Open

handrews wants to merge 3 commits into
OAI:mainfrom
handrews:cleanup/shared-md2html-fixtures

Conversation

@handrews

Copy link
Copy Markdown
Member

The migration to the build-infra package orphaned some repository-specific markdown tests. This relies on OAI/build-infra#65 to run the test out of that repository but with the existing OpenAPI Specification fixtures, preserving the correct division between generic infrastructure and spec-specific test data.

The first commit updates build-infra to pick up the changes on that side. The second adds the configuration on this side and replaces the local script with a build-infra import. The third removes a second spec.config.json file that was in the test fixture directory, as it is no longer needed (and was probably not ever a good idea due to confusion and risk of drift).

  • schema changes are included in this pull request
  • schema changes are needed for this pull request but not done yet
  • no schema changes are needed for this pull request

Refresh the #main resolution to include the shared downstream md2html fixture runner.\n\nCo-authored-by: Codex <codex@openai.com>
Keep the OpenAPI-specific rendering fixtures in this repository while delegating fixture discovery and exact-output comparison to build-infra.\n\nCo-authored-by: Codex <codex@openai.com>
Point the md2html fixtures at the production specification configuration and remove the redundant fixture-local copy.\n\nCo-authored-by: Codex <codex@openai.com>
@handrews
handrews requested review from a team as code owners October 14, 2026 23:59
@handrews handrews changed the title Cleanup/shared md2html fixtures Re-connect OAS-specific md2html tests Oct 14, 2026

This branch has not been deployed

No deployments
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.

1 participant