Skip to content

docs: dependency bump and fix failing pipeline - #484

Merged
alexluckett merged 4 commits into
mainfrom
chore/docusaurus-theme-govuk-v1
Oct 23, 2026
Merged

alexluckett merged 4 commits into
mainfrom
chore/docusaurus-theme-govuk-v1

Conversation

@alexluckett

@alexluckett alexluckett commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Proposed change

Bump defra-theme-govuk and fix the failing pipeline.

Jira ticket:

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Misc. (documentation, build updates, etc)

Checklist

  • You have executed this code locally and it performs as expected.
  • You have added tests to verify your code works.
  • You have added code comments and JSDoc, where appropriate.
  • There is no commented-out code.
  • You have added developer docs in README.md and docs/* (where appropriate, e.g. new features).
  • The tests are passing (npm run test).
  • The linting checks are passing (npm run lint).
  • The code has been formatted (npm run format).

forms-model 4 exports the output condition as an enum that holds a Joi
reference object. jsonschema2md cannot render an object enum value, so
schema doc generation crashed and the docs deploy failed. The enum is
removed because its allowed values come from the form's conditions
rather than a fixed list.
v1.0.0 adds react-router-dom ^5.3.4 as a peer dependency, so it is now
declared as a direct dev dependency next to react and react-dom.
The parameter type and runtime guard now narrow the value, so the cast
inside the function is no longer needed.
@alexluckett
alexluckett marked this pull request as ready for review October 22, 2026 22:43

@Scullyon Scullyon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

The schema docs script is over the 500-line limit that SonarCloud sets
for one file. The enum cleanup is a self-contained schema transform,
so it now has its own module.
@sonarqubecloud

Copy link
Copy Markdown

@alexluckett
alexluckett merged commit 6d65f81 into main Oct 23, 2026
23 checks passed
@alexluckett
alexluckett deleted the chore/docusaurus-theme-govuk-v1 branch October 23, 2026 10:45
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