Skip to content

docs/api-versioning: Mention changing enum types as an example of an incompatible change#5290

Merged
akshaymankar merged 1 commit into
developfrom
docs-change-enums
Jun 25, 2026
Merged

docs/api-versioning: Mention changing enum types as an example of an incompatible change#5290
akshaymankar merged 1 commit into
developfrom
docs-change-enums

Conversation

@akshaymankar

Copy link
Copy Markdown
Member

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d No changelog.
  • Read and follow the PR guidelines

@akshaymankar akshaymankar requested review from a team as code owners June 25, 2026 13:14
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jun 25, 2026

@supersven supersven 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.

LGTM 👍

(I'm wondering if we can't have automatized checks for this via swagger / the Servant type ... 🤔 However, this is more a topic for an upcoming Hackathon and not part of this PR's scope)

@akshaymankar

Copy link
Copy Markdown
Member Author

(I'm wondering if we can't have automatized checks for this via swagger / the Servant type ... 🤔 However, this is more a topic for an upcoming Hackathon and not part of this PR's scope)

Wondering the same, we shouldn't wait so long IMO. This can perhaps just be done as BAU work.

@akshaymankar akshaymankar merged commit 77597da into develop Jun 25, 2026
5 checks passed
@akshaymankar akshaymankar deleted the docs-change-enums branch June 25, 2026 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants