Skip to content

chore: prevent most common misconfigurations - #83

Merged
maximilianbraun merged 1 commit into
crossplane-contrib:mainfrom
christophrj:chore/add-xp-v2-registry-error-handling
Sep 18, 2025
Merged

maximilianbraun merged 1 commit into
crossplane-contrib:mainfrom
christophrj:chore/add-xp-v2-registry-error-handling

Conversation

@christophrj

@christophrj christophrj commented Aug 18, 2025 •

Copy link
Copy Markdown
Collaborator

see #80

Added some basic validation checks to fail fast with breaking v2 changes.
This is just for user convenience rather than a necessary change.

This could also be implemented in terms of documentation if you don't want to maintain any additional code.
Or by moving forward to v2 completely in removing the controllerconfig type and registry flag.

@christophrj
christophrj force-pushed the chore/add-xp-v2-registry-error-handling branch 5 times, most recently from 131eb7b to 4341c5f Compare August 20, 2025 17:43
@christophrj

christophrj commented Aug 21, 2025 •

Copy link
Copy Markdown
Collaborator Author

fixed e2e test so this can be merged independent of #82
in case #82 is merged first, the version pinning can be removed

@christophrj
christophrj force-pushed the chore/add-xp-v2-registry-error-handling branch from 4341c5f to d5b7b89 Compare August 25, 2025 09:19

@maximiliantech maximiliantech left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🫶

@maximilianbraun maximilianbraun left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Lgtm, Thanks 👌🖖

would you mind resolving the merge conflicts?

On-behalf-of: @SAP christopher.junk@sap.com
Signed-off-by: Christopher Junk <christopher.junk@sap.com>
@christophrj
christophrj force-pushed the chore/add-xp-v2-registry-error-handling branch from d5b7b89 to 5f377dc Compare September 18, 2025 11:27
@christophrj

Copy link
Copy Markdown
Collaborator Author

done

@maximilianbraun
maximilianbraun merged commit 7d9113d into crossplane-contrib:main Sep 18, 2025
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