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

The addition to #442 #465

Merged
merged 6 commits into from
Sep 7, 2016
Merged

The addition to #442 #465

merged 6 commits into from
Sep 7, 2016

Conversation

TikhomirovSergey
Copy link
Contributor

@TikhomirovSergey TikhomirovSergey commented Sep 6, 2016

Change list

  • gradle.build was improved.
  • the migration to java 8 has been started
  • AOP tools for the listening to events were synchronized with actual
    project state
  • pom.xml was moved to the archive directory. This file may be helpful
    for some time.

Types of changes

What types of changes are you proposing/introducing to Java client?

  • 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)

SrinivasanTarget and others added 3 commits August 26, 2016 23:22
version updates

Changes as per latest pom.xml

Added pom.xml and updated eclipse compiler dependency
- gradle.build was improved.
- the migration to java 8 has been started
- AOP tools for the listening to events were synchronized with actual
project state
- pom.xml was moved to the `archive` directory. This file may be helpful
for some time.
@TikhomirovSergey TikhomirovSergey added this to the 5.0.0 milestone Sep 6, 2016
@TikhomirovSergey TikhomirovSergey self-assigned this Sep 6, 2016
@SrinivasanTarget
Copy link
Member

@TikhomirovSergey Overall LGTM 👍 One minor change eclipse compiler version has to be updated to latest.

Does gradle build worked fine from both terminal and IDE?

Also codacy may not work properly now since PR has a merge commits.

@TikhomirovSergey TikhomirovSergey merged commit ef779eb into appium:master Sep 7, 2016
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