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

Udpate jdt.ls-specific stuff to EPL 2.0 #10595

Closed
tsmaeder opened this issue Jul 31, 2018 · 3 comments
Closed

Udpate jdt.ls-specific stuff to EPL 2.0 #10595

tsmaeder opened this issue Jul 31, 2018 · 3 comments
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed. target/branch Indicates that a PR will be merged into a branch other than master.

Comments

@tsmaeder
Copy link
Contributor

Our code in master has been updated to EPL 2.0. There are some files in the 5730_java_ls_poc branch that are not in master, and as such have not been updated. @skabashnyuk has said there is a script that does the update.
The update should also include code in che-ls-jdt and in our Theia/Java plugin repo.

@tsmaeder tsmaeder added kind/task Internal things, technical debt, and to-do tasks to be performed. target/branch Indicates that a PR will be merged into a branch other than master. team/languages labels Jul 31, 2018
@skabashnyuk
Copy link
Contributor

mvn license:format
but you need to build che-parent first, at least once with this pr eclipse-che/che-parent#66

@vrubezhny
Copy link
Contributor

The result of the execution of mvn license:format doesn't lead into appearing of any changed files neigher in 5730_java_ls_poc branch nor in master branch of che-js-jdt repository.

However we still have a bunch of the Eclipse Public License v1.0 appearances in java and xml files. The lists of such files don't differ too much between 5730_java_ls_poc and master branches...

@skabashnyuk it looks like not all the java and xml files are being processed by mvn license:format, f.i (in eclipse/che master) :

@skabashnyuk
Copy link
Contributor

@vrubezhny it can happen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed. target/branch Indicates that a PR will be merged into a branch other than master.
Projects
None yet
Development

No branches or pull requests

3 participants