Skip to content

Basic support of the Eclipse Ceylon language#10096

Merged
davidfestal merged 9 commits intoeclipse-che:masterfrom
davidfestal:ceylon-support-step1
Jun 21, 2018
Merged

Basic support of the Eclipse Ceylon language#10096
davidfestal merged 9 commits intoeclipse-che:masterfrom
davidfestal:ceylon-support-step1

Conversation

@davidfestal
Copy link
Copy Markdown
Contributor

What does this PR do?

This PR adds basic support of the Eclipse Ceylon language in Eclipse Che:

  • language description,
  • syntax highlighting,
  • project creation.

What issues does this PR fix or reference?

It refers no issue and is a new feature contribution

Release Notes

Basic support of the Eclipse Ceylon language in Eclipse Che with:

  • language description,
  • syntax highlighting,
  • project creation.

@codenvy-ci
Copy link
Copy Markdown

Can one of the admins verify this patch?

@benoitf benoitf added the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jun 18, 2018
@skabashnyuk
Copy link
Copy Markdown
Contributor

ci-test

@codenvy-ci
Copy link
Copy Markdown

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:10096
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@skabashnyuk
Copy link
Copy Markdown
Contributor

@davidfestal please let us know when this PR would be ready to review.

@davidfestal
Copy link
Copy Markdown
Contributor Author

ci-test

@codenvy-ci
Copy link
Copy Markdown

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:10096
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@davidfestal
Copy link
Copy Markdown
Contributor Author

ci-test

@codenvy-ci
Copy link
Copy Markdown

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:10096
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@davidfestal
Copy link
Copy Markdown
Contributor Author

please let us know when this PR would be ready to review.

@skabashnyuk according to the last build, it seems ready to review.

@davidfestal davidfestal self-assigned this Jun 19, 2018
<artifactId>license-maven-plugin</artifactId>
<configuration>
<excludes>
<exclude>**/ceylon.svg</exclude>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Who made this file? Who own a copyright for it?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's the extact same file as the one already added in this PR: #5844 (comment)

Also see this answer to a question from @benoitf : #5844 (comment)

<excludes>
<!-- The files don't need a license since they will
be added as templates inside the user projects -->
<exclude>src/main/resources/project/**</exclude>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can you put a license in comments of templates?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

IMHO generated code is unlicensed. Each user is making the choice of the license to use for their project.
Generator library is licensed but not why user find as its "project files"

<parent>
<artifactId>che-plugin-parent</artifactId>
<groupId>org.eclipse.che.plugin</groupId>
<version>6.7.0-SNAPSHOT</version>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

here need to set new version 6.8.0-SNAPSHOT

- language description,
- syntax highlighting,
- project creation.

Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
@davidfestal davidfestal force-pushed the ceylon-support-step1 branch from 0715968 to aa7f43f Compare June 20, 2018 12:42
@davidfestal
Copy link
Copy Markdown
Contributor Author

ci-test

@codenvy-ci
Copy link
Copy Markdown

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:10096
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

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

ci-test

@codenvy-ci
Copy link
Copy Markdown

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:10096
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@davidfestal davidfestal merged commit ae657b4 into eclipse-che:master Jun 21, 2018
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jun 21, 2018
@benoitf benoitf added this to the 6.8.0 milestone Jun 21, 2018
hbhargav pushed a commit to hbhargav/che that referenced this pull request Dec 5, 2018
Basic support of the Eclipse Ceylon language support, with the following features:
- language description,
- syntax highlighting,
- project creation.

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants