Skip to content

fix: niks3の参照先を上流に戻す#1300

Merged
ncaq merged 1 commit into
masterfrom
switch-niks3-to-upstream
Jun 28, 2026
Merged

fix: niks3の参照先を上流に戻す#1300
ncaq merged 1 commit into
masterfrom
switch-niks3-to-upstream

Conversation

@ncaq

@ncaq ncaq commented Jun 28, 2026

Copy link
Copy Markdown
Owner

私が懸念して伝えた内容が、
Mic92/niks3#426
で解決が試みられたので、
まず上流の解決策である、
こちらを試してみる。

私が懸念して伝えた内容が、
Mic92/niks3#426
で解決が試みられたので、
まず上流の解決策である、
こちらを試してみる。
@ncaq ncaq self-assigned this Jun 28, 2026
@ncaq ncaq added Type: Bug Bug or Bug fixes Type: Dependencies Dependency issues or Changes to dependency files labels Jun 28, 2026
@github-actions

Copy link
Copy Markdown

nvd diff: seminar (base: master)

Version changes

  • [U*] #1 niks3 1.6.1 -> 1.7.0
    Closure size: 4978 -> 4978 (55 paths added, 55 paths removed, delta +0, disk usage +66.1KiB).

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

niks3の参照先をフォークgithub:ncaq/niks3/dup-multipartから上流github:Mic92/niks3に戻す変更です。

フォークのdup-multipartブランチが回避策として対処していた懸念は、上流の Mic92/niks3#426 で正式に解決されています。flake.lockに固定された新リビジョンaa930845はそのPRのマージコミットそのものであり、フォークの意図する変更は上流に取り込まれています。クライアント側のCLIインターフェースに変更はなく、本リポジトリの利用箇所(niks3-push-ncaqや各NixOSモジュール)も設定変更不要です。

リグレッションリスクは低く、依存を上流に一本化する妥当な変更と評価します。

Review metadata
  • Reviewed commit: f9271c0
  • PR: #1300
  • kyosei: 3.5.0
  • kyosei-action: 2.2.5
  • Claude Code: 2.1.174
  • Model: claude-opus-4-8
  • Execution: GitHub Actions (run)

Comment thread flake.nix
@ncaq ncaq enabled auto-merge June 28, 2026 09:49
@ncaq ncaq merged commit 9523d59 into master Jun 28, 2026
10 checks passed
@ncaq ncaq deleted the switch-niks3-to-upstream branch June 28, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Bug Bug or Bug fixes Type: Dependencies Dependency issues or Changes to dependency files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant