Skip to content

apollo_deployments: expose jsonnet build() evaluator behind a testing feature#14686

Draft
nimrod-starkware wants to merge 1 commit into
nimrod/jsonnet/extract-base-layer-constantsfrom
nimrod/jsonnet/harness-expose-evaluator
Draft

apollo_deployments: expose jsonnet build() evaluator behind a testing feature#14686
nimrod-starkware wants to merge 1 commit into
nimrod/jsonnet/extract-base-layer-constantsfrom
nimrod/jsonnet/harness-expose-evaluator

Conversation

@nimrod-starkware

Copy link
Copy Markdown
Contributor

Split the test-only jsonnet module into jsonnet_eval (the reusable build()
evaluator) and jsonnet_tests (the parity/applicative helpers). Gate jsonnet_eval
on any(feature = "testing", test) and make jrsonnet optional regular deps enabled
by the new testing feature, so other crates (the integration-test harness) can
call jsonnet_eval::build_service_configs without pulling jrsonnet into the
default/production dependency graph.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

… feature

Split the test-only jsonnet module into jsonnet_eval (the reusable build()
evaluator) and jsonnet_tests (the parity/applicative helpers). Gate jsonnet_eval
on any(feature = "testing", test) and make jrsonnet optional regular deps enabled
by the new testing feature, so other crates (the integration-test harness) can
call jsonnet_eval::build_service_configs without pulling jrsonnet into the
default/production dependency graph.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@reviewable-StarkWare

Copy link
Copy Markdown

This change is Reviewable

This was referenced Jul 2, 2026

nimrod-starkware commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

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