Skip to content

build(jsonschema): build the schema generator in a go workspace - #471

Merged
kke merged 1 commit into
mainfrom
fix/jsonschema-module-workspace
Sep 14, 2026
Merged

kke merged 1 commit into
mainfrom
fix/jsonschema-module-workspace

Conversation

@kke

@kke kke commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

internal/jsonschema depends on the parent rig module through a local replace, so its indirect requirements have to match whatever the root go.mod requires. Every dependency bump in the root module leaves them stale and the go command then refuses to build here with "updates to go.mod needed" until this module is tidied as well. Dependabot never does that, so all of its pull requests fail schema-validation.

Add a go.work that covers the schema module and the repository root so the build list is computed across both, and ignore the generated go.work.sum - the per-module go.sum files already cover every selected version. The workspace file lives in internal/jsonschema so that root module commands keep running in plain module mode.

internal/jsonschema depends on the parent rig module through a local
replace, so its indirect requirements have to match whatever the root
go.mod requires. Every dependency bump in the root module leaves them
stale and the go command then refuses to build here with "updates to
go.mod needed" until this module is tidied as well. Dependabot never
does that, so all of its pull requests fail schema-validation.

Add a go.work that covers the schema module and the repository root so
the build list is computed across both, and ignore the generated
go.work.sum - the per-module go.sum files already cover every selected
version. The workspace file lives in internal/jsonschema so that root
module commands keep running in plain module mode.

Signed-off-by: Kimmo Lehto <klehto@mirantis.com>
@kke
kke merged commit 69d328c into main Sep 14, 2026
16 checks passed
@kke
kke deleted the fix/jsonschema-module-workspace branch September 14, 2026 12:46
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