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

再次发现几个问题 #7

Closed
wi1dcard opened this issue Nov 22, 2018 · 6 comments
Closed

再次发现几个问题 #7

wi1dcard opened this issue Nov 22, 2018 · 6 comments

Comments

@wi1dcard
Copy link
Collaborator

  1. 版本号是否应当采用语义化版本?
  2. Git 应当忽略 vendor 文件夹。
  3. 有一张图片无法加载。
    image
  4. Travis 脚本应当使用 composer install 而非 composer update
  5. composer.json 配置内的类型应当为 project,参考:https://docs.phpcomposer.com/04-schema.html#type
  6. 建议增加 php-cs-fixer 统一代码风格。
@louislivi
Copy link
Owner

1.可以的
2.为了使用者方便,下载即用 SMProxy本身没有依赖只是用来做自动加载,初次将vender 初次传上去了 后面vender是做了忽略的
3.图片是git缓存问题把图片点开一次就好了
4.因为git有composer.json 并有vender 所以使用的update
5.可以的
6.可以的

@wi1dcard
Copy link
Collaborator Author

wi1dcard commented Nov 22, 2018

2.为了使用者方便,下载即用 SMProxy本身没有依赖只是用来做自动加载,初次将vender 初次传上去了 后面vender是做了忽略的

你说的这种方法我认为不可取。

因为这样存储的自动加载文件并没有缓存,例如:

如果使用者并不执行 composer dump / composer update 等命令,则 Autoloader 加载任何资源都需要 Fall over 到查找文件系统,相比直接的 Autoload 缓存肯定慢了不少。

为了支持下载即用,我觉得更合适的方案是 Cut release 时,打包成 PHAR 并上传到 GitHub Release Assets,这样既能确保单文件、下载(非 git clone)即用,又能保证运行的一定是 stable,而非 dev / alpha / beta ...

@gitzwt
Copy link

gitzwt commented Nov 23, 2018

stop时smproxy进程没kill了,导致再次启动时3366端口占用无法启动,得手动kill

@gitzwt
Copy link

gitzwt commented Nov 23, 2018

2.为了使用者方便,下载即用 SMProxy本身没有依赖只是用来做自动加载,初次将vender 初次传上去了 后面vender是做了忽略的

你说的这种方法我认为不可取。

因为这样存储的自动加载文件并没有缓存,例如:

如果使用者并不执行 composer dump / composer update 等命令,则 Autoloader 加载任何资源都需要 Fall over 到查找文件系统,相比直接的 Autoload 缓存肯定慢了不少。

为了支持下载即用,我觉得更合适的方案是 Cut release 时,打包成 PHAR 并上传到 GitHub Release Assets,这样既能确保单文件、下载(非 git clone)即用,又能保证运行的一定是 stable,而非 dev / alpha / beta ...

这个建议很好,帮忙他一起完善维护吧. 已经使用在项目上,不希望dev的版本出问题

@wi1dcard
Copy link
Collaborator Author

@gitzwt @louislivi 这个仓库 tag 已经打乱了,提交信息也已经带了 vendor,只能给我直接推送的权限,不然不好搞。

@louislivi
Copy link
Owner

@gitzwt 已解决,已重构进程管理模块

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

No branches or pull requests

3 participants