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

add docker-compose.yml #51

Closed
wants to merge 1 commit into from
Closed

add docker-compose.yml #51

wants to merge 1 commit into from

Conversation

chunzhimoe
Copy link
Contributor

加了个docker-compose配置文件

@Jeric-X
Copy link
Owner

Jeric-X commented Mar 23, 2024

感谢贡献
请把文件放到src/SyncClipboard.Server目录中,并在README的docker部分添加如何使用docker compose的描述
关于Volume映射,可以注释掉使用默认值,当然写成/path/to/...也可以,但是不管怎么处理都需要在README里写明,至少让用户跟随README的操作后能够正常运行,并且知道如何修改默认配置

@feranwq
Copy link

feranwq commented Mar 25, 2024

建议内置默认配置, 允许通过环境变量覆盖配置, 大多数需求应该就是改密码or 端口

@Jeric-X
Copy link
Owner

Jeric-X commented Mar 25, 2024

建议内置默认配置, 允许通过环境变量覆盖配置, 大多数需求应该就是改密码or 端口

这暂时是项目本身的问题,现在还没有很好的修改配置的方法,可能以后会改
这个PR只要弄好最基本的就可以了

@chunzhimoe
Copy link
Contributor Author

感谢贡献 请把文件放到src/SyncClipboard.Server目录中,并在README的docker部分添加如何使用docker compose的描述 关于Volume映射,可以注释掉使用默认值,当然写成/path/to/...也可以,但是不管怎么处理都需要在README里写明,至少让用户跟随README的操作后能够正常运行,并且知道如何修改默认配置

不好意思,给您带来困扰了。晚上我回去加!

@chunzhimoe chunzhimoe closed this by deleting the head repository Mar 28, 2024
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

Successfully merging this pull request may close these issues.

3 participants