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

Ceylon stack #5844

Merged
merged 6 commits into from
Aug 7, 2017
Merged

Ceylon stack #5844

merged 6 commits into from
Aug 7, 2017

Conversation

davidfestal
Copy link
Contributor

What does this PR do?

It add a stack for the Ceylon language, supporting 3 backends: JVM, Javascript on Nodes, and Dart.
It also adds a Hello World sample in the samples.json file.

What issues does this PR fix or reference?

This doesn't reference any issue, since it's a feature proposal.

Changelog

Add a stack to support the Ceylon language on the JVM, NodeJS and Dart ptatforms

Release Notes

Add a stack to support the Ceylon language on the JVM, NodeJS and Dart ptatforms

Docs PR

Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
@codenvy-ci
Copy link

Can one of the admins verify this patch?

1 similar comment
@codenvy-ci
Copy link

Can one of the admins verify this patch?

},
{
"description": "Ceylon stack with Java, Javascript and Dart backends on CentOS.",
"creator": "David Festal",
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we use creator - "ide" here?

@ashumilova ashumilova added this to the 5.16.0 milestone Jul 30, 2017
@ashumilova ashumilova added the kind/enhancement A feature request - must adhere to the feature request template. label Jul 30, 2017
}
},
{
"description": "Ceylon stack with Java, Javascript and Dart backends on CentOS.",
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 small typo on JavaScript

"name": "JDK"
},
{
"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.

"name": "CentOS"
},
{
"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.

I think the version is higher

Copy link
Contributor

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

what is the license of the svg file ?

Signed-off-by: David Festal <dfestal@redhat.com>
@davidfestal
Copy link
Contributor Author

what is the license of the svg file ?

Creative Commons I think, but I've asked for formal confirmation. I'll keep you informed.

@davidfestal
Copy link
Contributor Author

@vparfonov do you need something more to be able to approve this PR (apart from the answer to the logo license question) ?

@davidfestal
Copy link
Contributor Author

@benoitf After consulting both the JBoss design team (which is the creator of the logo), and the Ceylon team), I confirm that the logo can be considered under the same license as the ceylon-lang website, which is the Creative Commons Attribution Share-Alike 3.0 (Unported) license as mentioned here.

The good news is that this license is one of the licenses approved by Eclipse for Non-Code, Example, and Other Content: https://eclipse.org/legal/noncodelicenses.php

Hope this will allow merging this PR.

@benoitf
Copy link
Contributor

benoitf commented Aug 7, 2017

@davidfestal maybe you should add license as headers of the SVG
Updating :

+<!-- Generator: Adobe Illustrator 15.0.2, SVG Export Plug-In . SVG Version: 6.00 Build 0)  -->

@riuvshin
Copy link
Contributor

riuvshin commented Aug 7, 2017

ci-build

"problems": [],
"source": {
"type": "git",
"location": "https://github.com/davidfestal/ceylon-hello-world",
Copy link
Contributor

Choose a reason for hiding this comment

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

is that hello world should be part of https://github.com/che-samples ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

would be great if it could. But @TylerJewell is the only member of this organization afaik.
OTH there are already quite a few samples whose git URL is from seom other place: https://github.com/tomitribe, https://github.com/kaloyan-raev, bitnami for example

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So is it required for the merge ?

Copy link
Contributor

Choose a reason for hiding this comment

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

@davidfestal no no not required for the merge. I also know that there are other examples
I can create "ceylon-hello-world" for you (or fork your repository on che-samples)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

forking would be good. Or simply creating the repo if I can have write access to it.

@codenvy-ci
Copy link

Build # 3294 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/3294/ to view the results.

@riuvshin
Copy link
Contributor

riuvshin commented Aug 7, 2017

@davidfestal please be aware that build fails with:

14:43:58 [WARNING] Missing header in: /home/codenvy/workspace/che-pullrequests-build/ide/che-core-ide-stacks/src/main/resources/stacks-images/type-ceylon.svg

to fix this please perform mvn license:format

@davidfestal
Copy link
Contributor Author

@benoitf @riuvshin I added the license into the svg file + changed the sample to point to https://github.com/che-samples/ceylon-hello-world

@riuvshin
Copy link
Contributor

riuvshin commented Aug 7, 2017

ci-build

Signed-off-by: David Festal <dfestal@redhat.com>
@davidfestal
Copy link
Contributor Author

@benoitf done

@benoitf
Copy link
Contributor

benoitf commented Aug 7, 2017

ci-build

@codenvy-ci
Copy link

@benoitf benoitf merged commit 2e6d797 into eclipse-che:master Aug 7, 2017
@davidfestal
Copy link
Contributor Author

@vparfonov do we need something else to merge ?

@riuvshin
Copy link
Contributor

riuvshin commented Aug 7, 2017

@davidfestal he is on PTO :)

dmytro-ndp pushed a commit to dmytro-ndp/che that referenced this pull request Aug 16, 2017
* Add a stack for Ceylon 1.3.2 with JVM, Javascript and Dart backends

Signed-off-by: David Festal <dfestal@redhat.com>
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
* Add a stack for Ceylon 1.3.2 with JVM, Javascript and Dart backends

Signed-off-by: David Festal <dfestal@redhat.com>
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.

5 participants