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

Maven to Gradle Migration #442

Merged
merged 1 commit into from
Sep 7, 2016
Merged

Maven to Gradle Migration #442

merged 1 commit into from
Sep 7, 2016

Conversation

SrinivasanTarget
Copy link
Member

Change list

Maven to Gradle Migration

Types of changes

What types of changes are you proposing/introducing to Java client?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Details

We will be using Gradle to build instead of Maven from our next release.Details are here:
#214

@SrinivasanTarget
Copy link
Member Author

@TikhomirovSergey FYI :)

@SrinivasanTarget
Copy link
Member Author

@bootstraponline You can also join this PR :)

}

dependencies {
ecj 'org.eclipse.jdt.core.compiler:ecj:4.4'
Copy link
Contributor

Choose a reason for hiding this comment

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

How about v4.5.1?

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure, i will update it.

version updates

Changes as per latest pom.xml

Added pom.xml and updated eclipse compiler dependency
@TikhomirovSergey
Copy link
Contributor

@SrinivasanTarget
There are problems with the importing/test running on IDE. I use IntellijIDEA 15.x and 2016.3/4. Please try to import the project.

PS: Don't forget to rename pom.xml to pom.xml_ for the history. This file may be helpful for some time

@SrinivasanTarget
Copy link
Member Author

Ok @TikhomirovSergey I will look into it today

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants