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

Update install-release.sh #1028

Closed
wants to merge 12 commits into from
Closed

Update install-release.sh #1028

wants to merge 12 commits into from

Conversation

sunshineplan
Copy link
Contributor

更新内容:

1.将所有exit改为return,并在适当的地方增加return,目的是增加脚本可读性
2.删除了不必要的return
3.修复部分缩进排版问题
4.检查新版本时增加检测本地是否安装的情况
5.copyFile功能简化
6.当/etc/v2ray/config.json存在时,不再生成/etc/v2ray文件夹
7.增加启动和停止服务失败的提醒
8.使用version参数时,不需要键入v,直接打版本号即可
9.少许逻辑及其他优化

@qingfengshi
Copy link

qingfengshi commented Apr 9, 2018 via email

@DarienRaymond
Copy link
Contributor

这个改动略大,我没有足够的把握review完所有的改动。如果能拆成几个小的RP分开提交,会有助于加快review的速度。

@sunshineplan
Copy link
Contributor Author

好的,我重新提交下。

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.

3 participants