Skip to content

#588 FIX #589

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Mar 6, 2017
Merged

#588 FIX #589

merged 3 commits into from
Mar 6, 2017

Conversation

TikhomirovSergey
Copy link
Contributor

Change list

  • update to Selenium 3.2.0
  • excessive dependency on guava was removed
  • source code synchronization

Types of changes

  • No changes in production code.
  • [x ] 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)

TikhomirovSergey and others added 3 commits March 5, 2017 23:43
@TikhomirovSergey
#588 FIX
35289a5 
- update to Selenium 3.2.0
- excessive dependency on guava was removed
- source code synchronization
@shs96c @TikhomirovSergey
Finish removing native events for Firefox
fa81997 
This was started, and the libraries no longer packaged
properly, but some of the bindings still required them.
@shs96c @TikhomirovSergey
Clean up import statements. No logical changes.
7cb73e7 
Done by running "optimize imports" in IJ CE.
@TikhomirovSergey TikhomirovSergey self-assigned this Mar 5, 2017
@jsf-clabot
Copy link

jsf-clabot commented Mar 5, 2017 •
edited
Loading

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ TikhomirovSergey
❌ shs96c

@TikhomirovSergey
Copy link
Contributor Author

@jaipaddy could you take a look at this PR too.

@TikhomirovSergey TikhomirovSergey merged commit ab258a2 into appium:master Mar 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SrinivasanTarget SrinivasanTarget SrinivasanTarget approved these changes

Assignees

@TikhomirovSergey TikhomirovSergey

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@TikhomirovSergey @jsf-clabot @SrinivasanTarget @shs96c