Skip to content
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

Spring Boot stack #4426

Merged
merged 3 commits into from
Mar 27, 2017
Merged

Spring Boot stack #4426

merged 3 commits into from
Mar 27, 2017

Conversation

gytis
Copy link
Contributor

@gytis gytis commented Mar 15, 2017

What does this PR do?

Adds the Spring Boot stack to the default Che assembly.

What issues does this PR fix or reference?

Changelog

Adds the Spring Boot stack to the default Che assembly.

Release Notes

We've added a new stack for the Spring Boot framework. You can now build Spring Boot applications by simply creating a workspace from the Spring Boot stack - no configuration necessary!

Docs PR

@gytis
Copy link
Contributor Author

gytis commented Mar 15, 2017

Recipe pull request: eclipse-che/che-dockerfiles#76

@codenvy-ci
Copy link

Can one of the admins verify this patch?

},
{
"name": "JDK",
"version": "1.8.0_45"
Copy link
Contributor

Choose a reason for hiding this comment

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

JDK is quite old there ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

I would drop the _45 as from the docker recipe we don't know the version

},
{
"name": "Maven",
"version": "3.2.2"
Copy link
Contributor

Choose a reason for hiding this comment

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

]
},
"stackIcon": {
"name": "type-java.svg",
Copy link
Contributor

Choose a reason for hiding this comment

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

question: logo is java but shouldn't it be spring boot logo ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do you use a designer for these images or should I find one online?

Copy link
Contributor

Choose a reason for hiding this comment

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

@gytis I'm happy to help you on that - no problem.
Could you tell me which icon do you wan to use?

Also, if you can point me to an SVG file, that would be ideal.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Since it's a Spring Boot stack, "power" button is more appropriate. Either green or black (like here https://spring.io/projects) up to you.

Gytis Trikleris and others added 2 commits March 20, 2017 12:12
Signed-off-by: Gytis Trikleris <[email protected]>
@slemeur
Copy link
Contributor

slemeur commented Mar 27, 2017

Added dedicated spring-boot icon

@slemeur slemeur requested a review from ashumilova March 27, 2017 10:52
<path d="M287.766184,130.123154 C294.116198,141.121701 294.113821,158.957986 287.766184,169.952417 L236.261619,259.160939 C229.911605,270.159486 214.463741,279.07557 201.768466,279.07557 L98.7593373,279.07557 C86.0593088,279.07557 70.6138211,270.155369 64.2661837,259.160939 L12.7616194,169.952417 C6.41160517,158.953869 6.41398201,141.117584 12.7616194,130.123154 L64.2661837,40.9146315 C70.6161979,29.9160842 86.0640625,21 98.7593373,21 L201.768466,21 C214.468494,21 229.913982,29.920201 236.261619,40.9146315 L287.766184,130.123154 Z M226.5498,145.751994 C226.5498,121.343483 215.299786,98.8434562 195.712709,84.1782601 C190.087703,79.8590586 181.951532,80.9639706 177.732777,86.6894238 C173.413576,92.3144305 174.618934,100.450601 180.243941,104.669356 C193.301992,114.513118 200.835483,129.479653 200.835483,145.751994 C200.835483,174.077921 177.732777,197.180627 149.406851,197.180627 C121.080924,197.180627 97.9782179,174.077921 97.9782179,145.751994 C97.9782179,129.479653 105.511709,114.513118 118.56976,104.669356 C124.194767,100.450601 125.400126,92.3144305 121.080924,86.6894238 C116.862169,80.9639706 108.826445,79.8590586 103.100992,84.1782601 C83.5139149,98.8434562 72.2639015,121.343483 72.2639015,145.751994 C72.2639015,188.240884 106.917961,222.894943 149.406851,222.894943 C191.895741,222.894943 226.5498,188.240884 226.5498,145.751994 Z M163.517823,72.3109722 C163.517823,65.5784093 157.123789,60 149.406851,60 C141.689913,60 135.295878,65.5784093 135.295878,72.3109722 L135.295878,143.441022 C135.295878,150.173585 141.689913,155.751994 149.406851,155.751994 C157.123789,155.751994 163.517823,150.173585 163.517823,143.441022 L163.517823,72.3109722 Z" id="spring-boot"></path>
</g>
</g>
</svg>
Copy link
Contributor

Choose a reason for hiding this comment

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

there is a missing newline

Signed-off-by: Stevan Le Meur <[email protected]>
@slemeur slemeur merged commit 2ddc27f into eclipse-che:master Mar 27, 2017
@gytis gytis deleted the master-spring-boot branch March 27, 2017 14:47
@slemeur slemeur added this to the 5.6.0 milestone Mar 28, 2017
@slemeur slemeur added the kind/enhancement A feature request - must adhere to the feature request template. label Mar 28, 2017
@JamesDrummond JamesDrummond mentioned this pull request Apr 2, 2017
9 tasks
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
* Spring Boot stack

Signed-off-by: Gytis Trikleris <[email protected]>

* Add dedicated icon for spring-boot stack

Signed-off-by: Stevan Le Meur <[email protected]>

* Add newline at end of svg file

Signed-off-by: Stevan Le Meur <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants