Skip to content

使っていないものを消す: GlitchTip、fork 用のレビュー、Renovate の古い規則 - #412

Merged
5ym merged 1 commit into
mainfrom
remove-unused
Oct 17, 2026
Merged

5ym merged 1 commit into
mainfrom
remove-unused

Conversation

@5ym

@5ym 5ym commented Oct 17, 2026

Copy link
Copy Markdown
Collaborator

簡素化の 1 本目(消すだけ)。

  • apps/_glitchtip/(541 行): _ で同期の対象外。クラスタにも DNS にも Entra にも無く、Renovate の PR を生むだけだった。戻すなら git の履歴から
  • .github/workflows/claude-code-review-fork.yml: 過去 20 回が全部 skipped、fork からの PR は 0 本
  • renovate.json: もう無い Mattermost / ERPNext の sourceUrl の規則。postgres を 17 に留める規則は forgejo と rybbit に要るので残し、説明だけ直した。sourceUrl の説明は netbird の規則に寄せた

git の外でやったこと: GitHub に登録だけ残っていた netbird-discover の workflow を無効化。クラスタに残っていた PVC adguardhome/adguardhome-ssl(#404 で git から消したが live に残り、gitops が OutOfSync だった)と、移行当日の final-* の Restore 23 個を消した。

🤖 Generated with Claude Code

https://claude.ai/code/session_01JbmWyUB9jcBcYCnNyiGBSV

@claude

claude Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Claude finished @5ym's task in 16s —— View job


レビュー結果

問題なし。 消すだけの PR として筋が通っています。

確認したこと:

  • glitchtip の参照は、README の「いまは無い」という履歴の 1 行だけです。マニフェスト、workflow、設定には残っていません。
  • bootstrap/argocd/repos.yaml の変更はコメントだけです。exclude: '{**/argocd.yaml,bootstrap/**,_*/**,**/_*/**}' は変わっていないので、_ で始まるディレクトリを同期しない挙動も変わりません。
  • renovate.json は JSON として整合しています。
    • 消したのは、もう無い Mattermost と ERPNext の sourceUrl の規則です。
    • postgres を <18 に留める規則は残っています。apps/ に forgejo と rybbit があるので、説明の書き換えも実態と合っています。
    • netbird の規則に sourceUrl の説明を寄せましたが、matchPackageNames、groupName、sourceUrl は変えていません。
  • README の命名規則の表から、今は無いアプリの例を外しています。

指摘ではありません。

  • .github/workflows/claude-code-review-fork.yml の削除は、内容を見ていないので判断できません。PR 本文の「過去 20 回が全部 skipped、fork からの PR は 0 本」を前提にしています。
  • bootstrap/argocd/helmchart.yaml などに残っている mattermost の文字列は、通知サービスの名前を意図的に残したものです。コメントにもそう書いてあり、この PR の範囲外です。
  • 本文の「git の外でやったこと」(workflow の無効化、PVC と Restore の削除)は、diff からは検証できません。

@5ym
5ym merged commit 5e3d1a3 into main Oct 17, 2026
2 checks passed
@5ym
5ym deleted the remove-unused branch October 17, 2026 08:33
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.

1 participant