Pixiv-daily-ranking-widget icon indicating copy to clipboard operation
Pixiv-daily-ranking-widget copied to clipboard

添加 docker 镜像

Open hujingnb opened this issue 2 years ago • 2 comments

为了方便布置, 将其打包为docker镜像已便于使用.

其中未完成内容:

  1. 当前仅实现了Dockerfile, 生成的镜像还没有推送到公共仓库
  2. 因镜像还没有公共名字, 故docker.md文档中, 镜像名称以xxx代替, 烦请更新后进行替换

hujingnb avatar Oct 07 '22 07:10 hujingnb

非常感谢你的贡献🤗不过还有几点需要优化的地方:

1、编码风格请参照 PSR 标准(PSR-1PSR-12
2、请尽量提供英文文档(实在不行就算了,我英文也不好 233)
3、docker.config.php 可以整合到 这里,无需增加一个独立文件
4、新 PR 请提给 develop 分支

mokeyjay avatar Oct 07 '22 15:10 mokeyjay

非常感谢你的贡献🤗不过还有几点需要优化的地方:

1、编码风格请参照 PSR 标准(PSR-1PSR-12) 2、请尽量提供英文文档(实在不行就算了,我英文也不好 233) 3、docker.config.php 可以整合到 这里,无需增加一个独立文件 4、新 PR 请提给 develop 分支

针对您提的几个问题.

  1. 我进行了简单修改. 主要就是将config.php文件中的方法单独拿到一个文件中. 不知道是否还有其他存在问题的地方.
  2. 英文文档我简单写了一下, 您可以过一下看是否有语法错误等问题(我英语课太差了)
  3. 您说的将环境变量的读取直接整合到Config.php文件中, 我个人认为不妥. 因为这样修改的话, 相当于直接在项目中进行容器化的支持了, 会对项目内容造成入侵. 再或者是我理解错了
  4. 分支已改

hujingnb avatar Oct 09 '22 01:10 hujingnb

拖了快 3 个月非常抱歉🙏再次感谢你的贡献!

mokeyjay avatar Dec 25 '22 13:12 mokeyjay