Skip to content

Conversation

@haarchri
Copy link
Member

Description of your changes

rename const Group/Version to CRDGroup/CRDVersion to unblock code-gen for ex. IAM issue

Fixes #1025
aws-controllers-k8s/code-generator#251

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

@haarchri haarchri force-pushed the feature/bump-ack-generator branch from 7896b67 to 64843e9 Compare December 20, 2021 19:17
@haarchri haarchri requested a review from muvaf December 20, 2021 19:23
Copy link

@AaronME AaronME left a comment

Choose a reason for hiding this comment

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

LGTM!

@haarchri
Copy link
Member Author

@AaronME @muvaf i would prefer to rebase & merge directly before we cut the release v0.23.0

@muvaf
Copy link
Member

muvaf commented Jan 26, 2022

@haarchri is there anything blocking this PR from being merged?

@haarchri
Copy link
Member Author

Think only your review - i can Check Last Time a rebase and can Go with it

Copy link
Member

@muvaf muvaf left a comment

Choose a reason for hiding this comment

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

@haarchri LGTM! Feel free to merge once any maintainer gives an LGTM unless you want a specific person to take a look 🙂

…up/CRDVersion to unblock code-gen for IAM Group issue

Signed-off-by: haarchri <[email protected]>
@haarchri haarchri force-pushed the feature/bump-ack-generator branch from b913c50 to b5cec69 Compare January 26, 2022 13:26
@haarchri haarchri merged commit 741d6e0 into crossplane-contrib:master Jan 26, 2022
@haarchri haarchri deleted the feature/bump-ack-generator branch January 26, 2022 13:42
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.

ack code-gen iam failed - Group redeclared in this block / other declaration of Group

3 participants