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

Fix Maven dependency manangement: Use ${che.version} instead of ${project.version} to allow projects with different version to use Che dependencies. #4752

Merged

Conversation

sunix
Copy link
Contributor

@sunix sunix commented Apr 10, 2017

What does this PR do?

Fix Maven dependency manangement: Use ${che.version} instead of ${project.version} to allow projects with different version to use Che dependencies.

What issues does this PR fix or reference?

Updates #1335

Changelog

Fix Maven dependency manangement: Use ${che.version} instead of ${project.version} to allow projects with different version to use Che dependencies.

Release Notes

Docs PR

…ersion} to allow projects with different version to use Che dependencies

Signed-off-by: Sun Seng David Tan <sutan@redhat.com>
@riuvshin
Copy link
Contributor

riuvshin commented Apr 10, 2017

@codenvy-ci
Copy link

@sunix
Copy link
Contributor Author

sunix commented Apr 10, 2017

@riuvshin thanks ... :) will fixup

…oject.version} to allow projects with different version to use Che dependencies

Signed-off-by: Sun Seng David Tan <sutan@redhat.com>
@codenvy-ci
Copy link

@sunix sunix merged commit 26d23b0 into eclipse-che:master Apr 11, 2017
@JamesDrummond JamesDrummond added this to the 5.8.0 milestone Apr 26, 2017
@JamesDrummond JamesDrummond added the target/branch Indicates that a PR will be merged into a branch other than master. label May 2, 2017
@sunix
Copy link
Contributor Author

sunix commented May 3, 2017

@JamesDrummond why target/branch label ?

@JamesDrummond JamesDrummond removed the target/branch Indicates that a PR will be merged into a branch other than master. label May 3, 2017
@JamesDrummond
Copy link
Contributor

My mistake.

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.

5 participants