Skip to content

Make global.json global again#4166

Merged
bfops merged 8 commits intomasterfrom
bfops/global-global-json
Jan 30, 2026
Merged

Make global.json global again#4166
bfops merged 8 commits intomasterfrom
bfops/global-global-json

Conversation

@bfops
Copy link
Collaborator

@bfops bfops commented Jan 29, 2026

Description of Changes

We have had several global.json files introduced that aren't symlinks to the root global.json.

This PR fixes that drift, and adds a CI check.

API and ABI breaking changes

None.

Expected complexity level and risk

2

Testing

  • cargo ci global-json-policy succeeds
  • cargo ci global-json-policy fails if you add a new global.json file somewhere
  • Rest of CI passes, confirming that templates are still working properly

bfops and others added 3 commits January 29, 2026 14:21
…ain permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Zeke Foppa <196249+bfops@users.noreply.github.com>
@bfops bfops marked this pull request as ready for review January 29, 2026 22:51
Copy link
Collaborator

@jdetter jdetter left a comment

Choose a reason for hiding this comment

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

An optional nit, this looks great. Thank Zeke 🙏

@bfops bfops added this pull request to the merge queue Jan 30, 2026
Merged via the queue into master with commit 6772f0c Jan 30, 2026
29 checks passed
@bfops bfops deleted the bfops/global-global-json branch February 6, 2026 03:23
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