search

mokeyjay / Pixiv-daily-ranking-widget

会自动更新的P站每日榜小挂件，适合放在博客侧边栏等地方 Pixiv daily ranking widget, automatically updates, suitable for blog sidebar, etc.
https://www.mokeyjay.com/archives/1063
MIT License
280 stars 37 forks source link

添加 docker 镜像 #19

Closed hujingnb closed 1 year ago

hujingnb commented 2 years ago

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

其中未完成内容:

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

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

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

hujingnb commented 2 years ago

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

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

针对您提的几个问题.

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

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