Pixiv-daily-ranking-widget
Pixiv-daily-ranking-widget copied to clipboard
添加 docker 镜像
为了方便布置, 将其打包为docker
镜像已便于使用.
其中未完成内容:
- 当前仅实现了
Dockerfile
, 生成的镜像还没有推送到公共仓库 - 因镜像还没有公共名字, 故
docker.md
文档中, 镜像名称以xxx
代替, 烦请更新后进行替换
非常感谢你的贡献🤗不过还有几点需要优化的地方:
1、编码风格请参照 PSR 标准(PSR-1、PSR-12)
2、请尽量提供英文文档(实在不行就算了,我英文也不好 233)
3、docker.config.php
可以整合到 这里,无需增加一个独立文件
4、新 PR 请提给 develop 分支
非常感谢你的贡献🤗不过还有几点需要优化的地方:
1、编码风格请参照 PSR 标准(PSR-1、PSR-12) 2、请尽量提供英文文档(实在不行就算了,我英文也不好 233) 3、
docker.config.php
可以整合到 这里,无需增加一个独立文件 4、新 PR 请提给 develop 分支
针对您提的几个问题.
- 我进行了简单修改. 主要就是将
config.php
文件中的方法单独拿到一个文件中. 不知道是否还有其他存在问题的地方. - 英文文档我简单写了一下, 您可以过一下看是否有语法错误等问题(我英语课太差了)
- 您说的将环境变量的读取直接整合到
Config.php
文件中, 我个人认为不妥. 因为这样修改的话, 相当于直接在项目中进行容器化的支持了, 会对项目内容造成入侵. 再或者是我理解错了 - 分支已改
拖了快 3 个月非常抱歉🙏再次感谢你的贡献!