-
Notifications
You must be signed in to change notification settings - Fork 21
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
added examples from pulumiUP #542
Conversation
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.
Looks good, just a couple of hints, and also changelog is missing.
<version>3.15.1</version> | ||
</dependency> | ||
|
||
<dependency> |
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.
nit: formatting issue, tabs vs spaces?
return; | ||
} | ||
|
||
public interface LambdaArgs { |
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.
nit: you can add @FunctionalInterface
https://docs.oracle.com/javase/8/docs/api/java/lang/FunctionalInterface.html
* Creates an EKS cluster | ||
* @param clusterPolicies | ||
*/ | ||
public EKSStack(List<Policy> clusterPolicies) { |
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.
nit: could have varargs overload
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.
true, but who knows what kinds of strange parameters are going to go into this function's constructor
public class HelloControllerIT { | ||
|
||
@Autowired | ||
private TestRestTemplate template; |
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.
formating issue
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 copy pasted this. I'm going to add a readme with attribution
|
||
public class PulumiStack { | ||
|
||
public static void main(String[] args) { |
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.
formatting issue
added link to base project
reverting changes
reverting changes
Description
Fixes #476
Checklist