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

リファクタ: gitignore標準化 #280

Closed
tarepan opened this issue Jan 8, 2022 · 2 comments · Fixed by #285
Closed

リファクタ: gitignore標準化 #280

tarepan opened this issue Jan 8, 2022 · 2 comments · Fixed by #285
Labels

Comments

@tarepan
Copy link
Contributor

tarepan commented Jan 8, 2022

内容

(discussed in #271)

現在のgitignoreはVOICEVOX独自の内容です.
GitHub公式のPython用gitignoreに追従すれば、ベストプラクティスを踏襲し、かつ今後の更新が楽になります.
gitignoreの置き換えを提案します.

Pros 良くなる点

  • 標準的Pythonプロジェクトとの一致
  • ベストプラクティス更新への追従性↑

実現方法

  1. 現行とGitHub公式のdiff
  2. 影響検討
  3. 置き換え

VOICEVOXのバージョン

all

OSの種類/ディストリ/バージョン

all

@tarepan
Copy link
Contributor Author

tarepan commented Jan 8, 2022

Diff

GitHub公式にないVV独自属性は以下になります.

  • *.dist
  • *.build
  • .venv-release
  • /licenses.json
  • /cache
  • /build: 類似したGitHub公式属性 - build/

おそらくビルド周りの属性です.
詳しい方のコメントがいただければ幸いです.
ある程度情報が集まり次第、上記属性をGitHub公式に足したものをPRとする予定です.

@Hiroshiba
Copy link
Member

*.dist*.build/build/cacheはnuitkaが出すディレクトリです。
licenses.jsongenerate_licenses.pyの生成物です。
↑これらはgitignoreに足しても良さそうに感じました。

.venv-releaseは昔1人でメンテしてたときの名残なので不要っぽいです。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants