Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix issue #998 #1317

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

Merged
merged 11 commits into from
Jul 13, 2019
Merged

fix issue #998 #1317

merged 11 commits into from
Jul 13, 2019

Conversation

saeed-rz
Copy link
Collaborator

I check the issue #998 and test it on simple API server, the issue belongs to "httpclient" and after update the "httpclient" version, the issue has fixed

@saeed-rz
Copy link
Collaborator Author

I have fix the issue in ConcurrentSkipList and update gradle, the Travis CI build failed because of java version.
can you change the CI? @smarek

@smarek smarek merged commit 9efecab into android-async-http:master Jul 13, 2019
@smarek
Copy link
Member

smarek commented Jul 13, 2019

Travis CI configuration is stored in file /.travis.yml, you can update it along in PR next time

Also next time you can squash your commits into one commit, or I'll have to squash them on merging PR.

I've submitted Travis and SDK compatibility fixes, so update your local development, so you base on latest master in next PR.

Thank you for your hard work

@saeed-rz
Copy link
Collaborator Author

I will take care of commits next time.
thanks for tips and update travis.yml.

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.

2 participants