Update flake inputs, optionally

[grahamc]

Description

Checklist

• Tested changes against a test repository
• Added or updated relevant documentation (leave unchecked if not applicable)
• (If this PR is for a release) Updated README to point to the new tag (leave unchecked if not applicable)

Summary by CodeRabbit

• Chores
  • Enhanced the update-downstream workflow to conditionally check for and update dependencies when specific project configurations are detected.

[netlify]

✅ Deploy Preview for detsys-ts-docs ready!

Name	Link
🔨 Latest commit	c5e495f
🔍 Latest deploy log	https://app.netlify.com/projects/detsys-ts-docs/deploys/698cb8ecfe843c000880b21b
😎 Deploy Preview	https://deploy-preview-137--detsys-ts-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

📝 Walkthrough

Walkthrough

A new conditional step is added to the update-downstream workflow that checks for a flake.nix file and runs nix flake update --commit-lock-file if present. This extends the downstream update process to handle Nix flake dependencies.

Changes

Cohort / File(s)	Summary
Workflow Configuration .github/workflows/update-downstream.yml	Adds "Update flake inputs" step that conditionally runs nix flake update --commit-lock-file when flake.nix exists in the target directory, executed after repository fork.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A workflow hops with newfound grace,
Checking flakes in every place,
When nix files bloom, update takes flight,
Dependencies dance in the workflow's light! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: adding optional flake input updating to the workflow.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch push-yowyrwpruvzr

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

.github/workflows/update-downstream.yml (2)

81-87: Minor indentation inconsistency.

Line 85 uses 3 spaces for indentation inside the if block, while the similar pattern at lines 92-96 uses 2 spaces. Consider aligning for consistency.

🔧 Suggested fix for consistent indentation

       - name: Update flake inputs
         working-directory: target
         run: |
           if [ -f ./flake.nix ]; then
-             nix flake update --commit-lock-file
+            nix flake update --commit-lock-file
           fi

---

84-86: Consider scope implications for downstream PRs.

Running nix flake update updates all flake inputs, not just those related to detsys-ts. This broadens the scope of downstream PRs and may introduce unrelated changes. If this is intentional (keeping flake inputs current alongside detsys-ts), consider updating the PR title/body template (lines 106-114) to mention flake input updates for clarity to downstream reviewers.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}