Skip to content

disallow adding or updating things in lib with update#6083

Merged
mitchellwrosen merged 11 commits intotrunkfrom
26-01-02-disallow-update-lib
Jan 19, 2026
Merged

disallow adding or updating things in lib with update#6083
mitchellwrosen merged 11 commits intotrunkfrom
26-01-02-disallow-update-lib

Conversation

@mitchellwrosen
Copy link
Member

@mitchellwrosen mitchellwrosen commented Jan 2, 2026

Overview

Fixes #5927

This PR makes update fail if the scratch file contains any additions of or updates to definitions in lib.*. It also warns on slurp, before update whether an update would fail for this reason.

Test coverage

I've added a transcript that demonstrates the feature.

@mergify
Copy link
Contributor

mergify bot commented Jan 8, 2026

⚠️ The sha of the head commit of this PR conflicts with #6089. Mergify cannot evaluate rules on this PR. ⚠️

@mitchellwrosen mitchellwrosen marked this pull request as ready for review January 16, 2026 00:38
@mitchellwrosen mitchellwrosen merged commit 41bec14 into trunk Jan 19, 2026
31 checks passed
@mitchellwrosen mitchellwrosen deleted the 26-01-02-disallow-update-lib branch January 19, 2026 15:16
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.

Disallow update from targeting definitions in lib

2 participants