-
Notifications
You must be signed in to change notification settings - Fork 125
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add new test/release pipelines for SDK V2 migration #605
base: main
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: michaelhtm The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good stuff @michaelhtm !
|
||
- name: auto-generate-controllers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure but it looks like there is an indentation issue here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also the job name is conflicting with name in line 2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i ran yq..it doesn't seem like there's indentation issues. Fixing the job name
- aws-sdk-go-v2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to run this for both aws-sdk-go-v2 and on branch tag?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feels like we'll have two jobs running on branch tag events
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looking further into this
1df36a1
to
9138659
Compare
Depending on which branch a PR is created, to test and release the controller using different versions of code-gen and runtime (whether the controller is generated with aws-sdk V1 or V2) * Test
9138659
to
fca94c1
Compare
Description of changes:
Depending on which branch a PR is created, to test and release
the controller using different versions of code-gen and runtime
(whether the controller is generated with aws-sdk V1 or V2)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.