Skip to content

feat(common): backwards compatibility for strategic metadata#116

Merged
OpsBotPrime merged 3 commits into
masterfrom
vince/strategic-metadata-backwards-compatibility
Jul 7, 2026
Merged

feat(common): backwards compatibility for strategic metadata#116
OpsBotPrime merged 3 commits into
masterfrom
vince/strategic-metadata-backwards-compatibility

Conversation

@vincevannoort

@vincevannoort vincevannoort commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Description

This pull request makes the recently added strategic metadata backwards compatible, such that when versions of producers and consumers are not updated yet, they don't cause errors.

Note: with help from my friend Opus 4.8

@Qqwy Qqwy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I see one small way to improve the test(s) for this, but other than that I think this is a nice fix for the problem 😊 !

Comment thread opsqueue/src/common/submission.rs Outdated
Comment thread opsqueue/src/common/submission.rs
@vincevannoort vincevannoort marked this pull request as ready for review July 7, 2026 08:47
@vincevannoort

Copy link
Copy Markdown
Contributor Author

@OpsBotPrime merge and tag

Approved-by: vincevannoort
Priority: Normal
Auto-deploy: false
@OpsBotPrime

Copy link
Copy Markdown
Contributor

Rebased as 54172c3, waiting for CI …

@OpsBotPrime

Copy link
Copy Markdown
Contributor

CI job 🟡 started.

@OpsBotPrime

Copy link
Copy Markdown
Contributor

@vincevannoort I tagged your PR with v54. Please wait for the build of 54172c3 to pass and don't forget to deploy it!

@OpsBotPrime OpsBotPrime merged commit 54172c3 into master Jul 7, 2026
6 of 7 checks passed
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.

3 participants