Skip to content

Conversation

SherlockRen
Copy link

统一拼接链接方法,函数中也添加了相应的方法,方便使用。

@longbai
Copy link
Contributor

longbai commented Jul 15, 2016

十分感谢,麻烦处理下CI

*
* @link http://developer.qiniu.com/code/v6/api/kodo-api/image/imageview2.html
*/
final class ImageProcess
Copy link
Contributor

Choose a reason for hiding this comment

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

类名 建议 改为ImageUrlBuilder

Copy link
Author

Choose a reason for hiding this comment

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

没问题,总觉得放在Processing目录有点怪怪的,有没有更好的建议

Copy link
Contributor

Choose a reason for hiding this comment

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

只能这里了,属于数据处理业务

@SherlockRen
Copy link
Author

done

@SherlockRen
Copy link
Author

image
CI 修改完了 5.5自动测试的时候 http://api.qiniu.com/pfop 502 了

@longbai
Copy link
Contributor

longbai commented Jul 15, 2016

我重启下ci 看看

@longbai
Copy link
Contributor

longbai commented Jul 15, 2016

LGTM

@longbai longbai merged commit f27b219 into qiniu:master Jul 15, 2016
@SherlockRen
Copy link
Author

嗨 可否更新一下版本号 我现在想用新增的imageurlbuilder

在 2016年7月15日,下午6:04,Bai Long notifications@github.com 写道:

Merged #188 #188.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub #188 (comment), or mute the thread https://github.com/notifications/unsubscribe-auth/AGoFkXVguEn2YgYu8hVj1l9uH4SrRSLvks5qV1tGgaJpZM4JMVdN.

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