nixpkgs レビューログ

NixOS/nixpkgsのレビュー時に見逃していた・気を付けるべき点について記録しておく。

習慣

  • できるだけ起床してから30〜60分間はnixpkgsへのレビューに充てる
    • 朝の方が当日中に他のレビュワーやコントリビュータからの返信に対応しやすい
  • バージョン更新だけなど、レビューしやすいPRは作業の合間に取り組みたい...
    • まだnixpkgs-reviewが実行されていないPRに対して自動で実行しておくツールが欲しい
  • 過去にレビューしたPRの更新や、レビュワーから指摘にはできるだけ優先的に対応する
    • これが苦手なのが課題

一般

  • Approveしてから他のレビュワーによって改善点が指摘された場合にはメモしておく
  • Request Changesを送ってからマージされるまではTodoistに入れて確認するようにする
  • Approveしてからコミッタにマージされない場合はNixOS Discourseのprs-ready-for-reviewに投稿する
  • Request Changesを送る際にはできるだけ丁寧に改善点について説明する

ログ

2025/06/10

2025/06/09

2025/06/08

2025/06/07

2025/06/06

2025/06/05

2025/06/04

  • ocamlPackages.wasm: 2.0.1 -> 2.0.2
    • opamパッケージとしてビルドされているが、upstreamがduneプロジェクトの場合にはbuildDunePackageを用いて書き直した方が良い
  • ocamlPackages.ffmpeg: 1.2.1 -> 1.2.5
    • meta.changelogは、テキスト形式の方が扱いやすいので、https://raw.githubusercontent.com/で指定した方が良い
  • ocamlPackages.xenstore: 2.3.0 -> 2.4.0 (draft)
    • xenstore-toolがバージョンアップによって消されたAPIを使っていて、かつ報告場所がないので困っている、という旨のコメントをOCamlパッケージをメンテナンスしてる方に投げてみた
  • 30分くらいのつもりだったけど2時間経っていた
  • SATySFi
    • satysfiDirs.ml
    • ここで~/.opam/配下のパッケージパスを特定している
    • SATYSFI_RUNTIMEを書き換えると出力先を選べる
    • Pythonスクリプト不要かも。satyrographosに処理を委譲できそう