search

y-young / f1music

校园音乐征集投票系统 A system for electing annual school music
MIT License
9 stars 3 forks source link

修改env模板和部署脚本 #13

Closed acha666 closed 1 year ago

acha666 commented 1 year ago

合并了.env文件的模板，并对部署脚本做相应修改 将Markdown文档移动到仓库wiki 修复docker-compose.yml中Dockerfile目录错误

疑问：为何docker-compose.yml第五行同时存在image和build项，我注释了image

y-young commented 1 year ago

合并了.env文件的模板，并对部署脚本做相应修改 修复docker-compose.yml中Dockerfile目录错误

.env 的模板没必要合并吧？另外， refactor/laravel 分支应该修复了 docker-compose.yml 的问题，也有新的 docker 部署脚本

疑问：为何docker-compose.yml第五行同时存在image和build项，我注释了image

如果 docker-compose up 时不指定 --build 参数则拉取预构建镜像部署，否则根据 Dockerfile 本地构建

acha666 commented 1 year ago

我是觉得几个部署脚本统一用一个模板更合适 没看其他分支改重了真是抱歉（

-------- 原始信息 --------由： Googleplex @.> 日期: 2023/3/17 14:09 (GMT+08:00) 收件人： y-young/f1music @.> 抄送： Acha @.>, Author @.> 主题： Re: [y-young/f1music] 修改env模板和部署脚本 (PR #13)

合并了.env文件的模板，并对部署脚本做相应修改

修复docker-compose.yml中Dockerfile目录错误

.env 的模板没必要合并吧？另外， refactor/laravel 分支应该修复了 docker-compose.yml 的问题，也有新的 docker 部署脚本

疑问：为何docker-compose.yml第五行同时存在image和build项，我注释了image

如果 docker-compose up 时不指定 --build 参数则拉取预构建镜像部署，否则根据 Dockerfile 本地构建

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.> [ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "https://github.com/y-young/f1music/pull/13#issuecomment-1473198458", "url": "https://github.com/y-young/f1music/pull/13#issuecomment-1473198458", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { @.***": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

y-young commented 1 year ago

我是觉得几个部署脚本统一用一个模板更合适

其实应该改一下 deploy.sh，但是这个脚本太老了我不想动它了（

没看其他分支改重了真是抱歉（

那个分支之前重构完还没彻底测试过，有空的话可以帮忙测一下