-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
IDEX-3913: Implement stack api #351
Conversation
} | ||
|
||
@PostConstruct | ||
public void start() { |
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.
Use lock for @PostConstruct
and @PredDestroy
methods as well
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.
OK
847cad4
to
d719636
Compare
<artifactId>che-core-ide-stacks</artifactId> | ||
<packaging>jar</packaging> | ||
<name>Che Core :: IDE :: STACKS</name> | ||
</project> |
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.
missing empty line
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.
Ok
d719636
to
8c70828
Compare
@@ -0,0 +1,710 @@ | |||
[ |
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.
why these stacks are part of ide core module ? as it's more linked to the core/platform than IDE
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.
8c70828
to
98362d1
Compare
{ | ||
"name": "default", | ||
"limits": { | ||
"memory": 1000 |
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.
This field now called "ram"
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.
Ok, I'll fix it.
0b9a0e2
to
a4d32ca
Compare
+1 |
ok |
Generally it looks okay |
Signed-off-by: Aleksandr Andrienko <aandrienko@codenvy.com>
a4d32ca
to
fa7eb17
Compare
IDEX-3913: Implement stack api
_1 Upvote_ Signed-off-by: Aleksandr Andrienko aandrienko@codenvy.com