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

compose.ymlが追跡対象外になっていない? #14344

Closed
1 task done
t1nyb0x opened this issue Jul 31, 2024 · 7 comments · Fixed by #14345
Closed
1 task done

compose.ymlが追跡対象外になっていない? #14344

t1nyb0x opened this issue Jul 31, 2024 · 7 comments · Fixed by #14345
Labels
✨Feature This adds/improves/enhances a feature

Comments

@t1nyb0x
Copy link
Collaborator

t1nyb0x commented Jul 31, 2024

Summary

docker-compose.ymlがcompose.ymlに置き換わったと思うのですが、今までignoreされていたdocker-compose.ymlが消されcompose.ymlが設定されていないように見えます。

https://github.com/misskey-dev/misskey/blob/develop/.gitignore#L33-L39

そのため、compose.ymlを追跡対象外に設定したほうが良いと考えています

Purpose

docker-compose.ymlから推奨されているファイル名であるcompose.ymlに変更されていますが、
compose.ymlが追跡対象外から外れてしまっています。
また、docker-compose.ymlも追跡対象から外されているため、今までこのファイルでMisskeyコンテナを立てていたユーザーだと追跡対象としてコミットできてしまいます。

元々の設定を考えると、docker-compose.ymlとcompose.ymlは両方とも追跡対象外であるべきだと思うのですがいかがでしょうか?

Do you want to implement this feature yourself?

  • Yes, I will implement this by myself and send a pull request
@t1nyb0x t1nyb0x added the ✨Feature This adds/improves/enhances a feature label Jul 31, 2024
@anatawa12
Copy link
Member

うーん環境依存で存在するファイルを.gitignoreにいれるべきかは怪しいと思う。

(git add .なんかするなという話な気がしている)

@t1nyb0x
Copy link
Collaborator Author

t1nyb0x commented Jul 31, 2024

exampleがあって、そこからコピーをかけて自分用のcompose.ymlを作るということを考えると除外されていたほうが良いように見えますが違うってことですよね・・・

元々あった設定を外すということを考えるとどういう意図でignoreされなくなったのかは気になります。

@anatawa12
Copy link
Member

https://github.com/misskey-dev/misskey/commits/develop/.devcontainer/docker-compose.yml

あとはこのタイミングで削除が走ってるので.devcontiner/docker-compose.ymlは残らないと思います

@t1nyb0x
Copy link
Collaborator Author

t1nyb0x commented Jul 31, 2024

.devcontainer/docker-compose.yml ではなく、 ルート配下の docker-compose.yml のことを気にしていました。

サーバーを運用する際、ルート配下の docker-compose.ymldocker-compose.example.yml からコピーして使っていたので・・・。

@anatawa12
Copy link
Member

anatawa12 commented Jul 31, 2024

あー、リンク先がdevcontainerの話ししてたのでそっちの話だと思ってました。

なるほどです。ルートはあっても良さそう。(コピーして作る話だからなくても良いとは思うけど)

@t1nyb0x
Copy link
Collaborator Author

t1nyb0x commented Jul 31, 2024

とりあえずPR作ってみました。
必要であれば確認お願いいたします。

@tai-cha
Copy link
Contributor

tai-cha commented Aug 6, 2024

別issueでもいいものの #14096.dockerignoreからもcompose.ymlが消えているかも

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants