-
Notifications
You must be signed in to change notification settings - Fork 368
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
feat(ec2/securitygroup)!: Make region a required field #2003
Conversation
23add17
to
e69ebe1
Compare
/test-examples |
@haooliveira84 can you rebase on the latest master to fix e2e tests? |
f1809d4
to
dc9db4c
Compare
@haooliveira84 can you squash your changes into a single commit and sign it? |
dc9db4c
to
b6c3b49
Compare
Done @MisterMX |
It looks like something has gone wrong and a bunch of other changes has been included as well. @haooliveira84 can you check again? |
Done @MisterMX |
481606c
to
e3d843c
Compare
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.
The branch still contained a bunch of other commits. Probably because of the merge from master (better use rebase for that). I took the liberty to rebase the branch on the latest master and squash it into a single commit.
While doing that I noticed one thing that needs to be fixed before merge.
9b5c908
to
8d66ef4
Compare
5f048da
to
c169d27
Compare
d7de6c5
to
ba1165d
Compare
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.
LGTM. Thank you very much @haooliveira84!
Signed-off-by: Henrique Oliveira <henrique.antonio@grupoboticario.com.br> Signed-off-by: Maximilian Blatt (external expert on behalf of DB Netz) <maximilian.blatt-extern@deutschebahn.com> Signed-off-by: Henrique Oliveira <henrique.antonio@grupoboticario.com.br>
Description of your changes
Fixes #2002
I have:
make reviewable test
to ensure this PR is ready for review.How has this code been tested