Skip to content
This repository was archived by the owner on Aug 17, 2025. It is now read-only.

feat: breaking change detection#5085

Merged
alecthomas merged 1 commit intomainfrom
aat/breaking-changes
Jun 2, 2025
Merged

feat: breaking change detection#5085
alecthomas merged 1 commit intomainfrom
aat/breaking-changes

Conversation

@alecthomas
Copy link
Copy Markdown
Collaborator

This is intended for use with external API boundaries, including values that are being persisted to storage.

Currently this disallows:

  • Removing (or renaming) fields in data structures.
  • Changing a type.
  • Adding or removing enum values - adding is disallowed because a consumer that doesn't expect the new value will fail. Not ideal.

@alecthomas alecthomas force-pushed the aat/breaking-changes branch 5 times, most recently from add7ed1 to 5cebb3f Compare April 4, 2025 20:53
@alecthomas alecthomas mentioned this pull request Apr 4, 2025
@alecthomas alecthomas force-pushed the aat/breaking-changes branch 6 times, most recently from 96b0290 to d3baf99 Compare June 2, 2025 09:23
@alecthomas alecthomas marked this pull request as ready for review June 2, 2025 09:32
@alecthomas alecthomas requested a review from a team as a code owner June 2, 2025 09:32
@alecthomas alecthomas requested review from jvmakine and matt2e and removed request for a team June 2, 2025 09:32
This is intended for use with external API boundaries, including values that
are being persisted to storage.

Currently this disallows:

- Removing (or renaming) fields in data structures.
- Changing a type.
- Adding or removing enum values - adding is disallowed because a consumer that doesn't expect the new value will fail. Not ideal.
@alecthomas alecthomas force-pushed the aat/breaking-changes branch from d3baf99 to 98e7573 Compare June 2, 2025 09:42
@alecthomas alecthomas merged commit 7174eee into main Jun 2, 2025
110 checks passed
@alecthomas alecthomas deleted the aat/breaking-changes branch June 2, 2025 09:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant