Skip to content

Conversation

@NickLarsenNZ
Copy link
Member

This improves DX by emitting an error when invalid items are added to the versioned module:

image

@NickLarsenNZ NickLarsenNZ self-assigned this Jun 18, 2025
@NickLarsenNZ NickLarsenNZ moved this to Development: Waiting for Review in Stackable Engineering Jun 18, 2025
Copy link
Member

@maltesander maltesander left a comment

Choose a reason for hiding this comment

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

LGTM!

@NickLarsenNZ NickLarsenNZ added this pull request to the merge queue Jun 23, 2025
@NickLarsenNZ NickLarsenNZ moved this from Development: Waiting for Review to Development: Done in Stackable Engineering Jun 23, 2025
Merged via the queue into main with commit ecd20bd Jun 23, 2025
8 checks passed
@NickLarsenNZ NickLarsenNZ deleted the fix/stackable-versioned-error-item branch June 23, 2025 07:47
@sbernauer
Copy link
Member

Nice, thanks!
A changelog entry would be nice, especially as this is technically a breaking change

@NickLarsenNZ
Copy link
Member Author

NickLarsenNZ commented Jun 23, 2025

Nice, thanks! A changelog entry would be nice, especially as this is technically a breaking change

Ah yeah, I missed the changelog, but I don't see how it is a breaking change.

It was broken (because any unsupported code defined there was thrown away, and therefore unused/unusable). Now we get a warning/error.

Edit: I guess the breaking part is that unsupported items accidentally added in the module can now cause a compilation failure.

Done in #1062

@NickLarsenNZ NickLarsenNZ changed the title fix(stackable-versioned): Emit an error when invalid items are added to the versioned module fix(stackable-versioned)!: Emit an error when invalid items are added to the versioned module Jun 23, 2025
@sbernauer
Copy link
Member

exactly, previously compiling code is now not compiling any more.
Many thanks, approved!

@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants