Feature/support okhttp4 #346
Conversation
hamberluo
commented
Nov 1, 2019
hamberluo
commented
- 适配 gradle 到最新版本
- 适配 android test 依赖
- 适配 okhttp4
- 修改 README 和 tag
JemyCheung
left a comment
JemyCheung
left a comment
There was a problem hiding this comment.
非常感谢您的pr，okhttp4.x的兼容也是在我们考虑解决的范围内
| } | ||
| return null; | ||
| } | ||
|
|
There was a problem hiding this comment.
在3.+版本应该有个public的String_format方法，既然是public建议保留
There was a problem hiding this comment.
public 其实你们并没有使用到，我的本意是希望不要添加这个多余的类。
| private HttpDate() { | ||
| } | ||
| } | ||
|
|
There was a problem hiding this comment.
这个类可以放到package:com.qiniu.android.http下面
如果没有特殊使用需求，这个直接用okhttp的HttpDate
There was a problem hiding this comment.
okhttp4 的 HttpDate 已经删掉了，取而代之的类是 internal 的
| defaultConfig { | ||
| //applicationId "com.qiniu.android" | ||
| minSdkVersion 9 | ||
| minSdkVersion 15 |
There was a problem hiding this comment.
这个地方的变动我们希望它会是安全的，在更低版本的设备(非手机，有些老设备会集成sdk上传信息)我们希望能用
There was a problem hiding this comment.
这个变动是配合 gradle 升级的，否则会 build 不过，它不是一个独立的部分，而且我认为已经没有必要再支持 4.0 以下的手机
| } | ||
| dependencies { | ||
| classpath 'com.android.tools.build:gradle:2.1.0' | ||
| classpath 'com.android.tools.build:gradle:3.5.1' |
There was a problem hiding this comment.
配置2.1.0如果不影响代码release的话(您pr的代码)还是配置2.1.0；
本地环境3.5.1可以往2.1.0兼容不影响使用
但是这里一旦升高后就要要求其他开发人员强行升级高版本了
There was a problem hiding this comment.
不涉及插件的话，使用七牛 sdk 的人员不用关心这里的版本的，老的插件不适用于 gradle 的高级版本，为什么升 gradle，因为 gradle 本身迭代非常快，老版本的 gradle 有很多性能和其他问题。
|
Thanks for the PR |
3 participants
