FH-1031: Add ISO download URLs to README

[lucperkins]

Summary by CodeRabbit

• Documentation
  • Added a “You can download the ISOs at these addresses” section with a table linking x86 and aarch64 ISOs.
  • Replaced a plain note with a Markdown admonition for improved visibility.
  • Clarified installer image behavior: Flakes enabled by default and NetworkManager used instead of WPA Supplicant.
  • Expanded installed-tools list to document Determinate Nix (with login utility) and the FlakeHub CLI (fh) with usage examples.

[coderabbitai]

Walkthrough

Documentation updates in README.md: added an ISO download section with a two-row table for x86 and aarch64 ISOs; replaced a plain note with a Markdown admonition describing installer image differences (Flakes enabled via Determinate Nix; NetworkManager used); expanded installed-tools list to document Determinate Nixd, show a determinate-nixd login example, and add the fh CLI with an fh apply nixos example.

Changes

Cohort / File(s)

Summary of Changes

Docs: README content and formatting README.md

Added "You can download the ISOs" table (x86 Linux, aarch64 Linux). Replaced plain note with a Markdown admonition describing two differences: Flakes enabled by default via Determinate Nix and NetworkManager enabled. Expanded "installed tools" section to document Determinate Nix and determinate-nixd login example, and added fh CLI with an fh apply nixos usage example. No code, logic, or public API changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit peeks at README lines,
Adds ISO links and clearer signs. 🐇
Flakes and login, tools in view,
FH commands hop through.
Docs polished, small and true.

Pre-merge checks and finishing touches

✅ 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 "FH-1031: Add ISO download URLs to README" succinctly and accurately describes the primary change in this PR — adding ISO download URLs and related README content. It is concise, specific, and directly reflects the file-level changes (new section and table of ISO URLs), so a teammate scanning history will understand the main intent. Including the FH-1031 ticket prefix improves traceability without reducing clarity.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch provide-iso-urls

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

README.md (1)

38-39: Pluralize and tighten the NOTE; use upstream tool name casing.

Clarifies scope (both images), avoids “identical” ambiguity, and uses “wpa_supplicant”.

Apply this diff:

-> [!NOTE]
-> This NixOS installer image is identical to NixOS's ISOs, but with flakes enabled by default via Determinate Nix and NetworkManager enabled instead of bare WPA Supplicant.
+> [!NOTE]
+> These installer images are functionally identical to the upstream NixOS ISOs, with two differences:
+> - flakes are enabled by default via Determinate Nix
+> - NetworkManager is enabled instead of wpa_supplicant

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b281254 and f2367ad.

📒 Files selected for processing (1)

• README.md (2 hunks)

🧰 Additional context used

🪛 markdownlint-cli2 (0.17.2)

README.md

15-15: Bare URL used

(MD034, no-bare-urls)

---

16-16: Bare URL used

(MD034, no-bare-urls)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: build (UbuntuLatest32Cores128G, x86_64-linux)
• GitHub Check: build (UbuntuLatest32Cores128GArm, aarch64-linux)

🔇 Additional comments (1)

README.md (1)

11-17: Replace bare URLs with Markdown links (fixes MD034) — add reference-style defs and verify redirects.

Quick check: both URLs returned HTTP 307 (temporary redirect) — follow redirects (curl -L) or use the final destination URLs in the refs before merging.

Apply this diff:

-You can download the ISOs at these addresses:
+You can download the ISOs at these addresses:

-| System        | URL                                                                |
-| :------------ | :----------------------------------------------------------------- |
-| x86 Linux     | https://install.determinate.systems/nixos-iso/stable/x86_64-linux  |
-| aarch64 Linux | https://install.determinate.systems/nixos-iso/stable/aarch64-linux |
+| System        | URL             |
+| :------------ | :-------------- |
+| x86 Linux     | [Download][iso-x86_64]  |
+| aarch64 Linux | [Download][iso-aarch64] |

Add the link references near the other refs at the bottom:

+[iso-x86_64]: https://install.determinate.systems/nixos-iso/stable/x86_64-linux
+[iso-aarch64]: https://install.determinate.systems/nixos-iso/stable/aarch64-linux

Optionally add checksum links if available.

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

README.md (1)

38-41: Tighten wording, fix pluralization, and standardize punctuation/terminology.

Use plural “images,” reference “upstream NixOS,” add periods to list items, and use “wpa_supplicant.”

-> [!NOTE]
-> This NixOS installer image is identical to NixOS's ISOs, but with two important differences:
-> 1. Flakes are enabled by default via Determinate Nix
-> 2. NetworkManager is enabled instead of bare WPA Supplicant.
+> [!NOTE]
+> These installer images are identical to upstream NixOS ISOs, with two important differences:
+> 1. Flakes are enabled by default via Determinate Nix.
+> 2. NetworkManager is enabled instead of wpa_supplicant.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f2367ad and 877c4b3.

📒 Files selected for processing (1)

• README.md (2 hunks)

🧰 Additional context used

🪛 markdownlint-cli2 (0.17.2)

README.md

15-15: Bare URL used

(MD034, no-bare-urls)

---

16-16: Bare URL used

(MD034, no-bare-urls)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: build (UbuntuLatest32Cores128G, x86_64-linux)
• GitHub Check: build (UbuntuLatest32Cores128GArm, aarch64-linux)

🔇 Additional comments (1)

README.md (1)

11-17: Replace bare URLs with reference-style links (fixes MD034)

README.md lines 11–17 — Verified both URLs return HTTP 307 (redirect); apply this diff.

-You can download the ISOs at these addresses:
+You can download the ISOs at these addresses:

-| System        | URL                                                                |
-| :------------ | :----------------------------------------------------------------- |
-| x86 Linux     | https://install.determinate.systems/nixos-iso/stable/x86_64-linux  |
-| aarch64 Linux | https://install.determinate.systems/nixos-iso/stable/aarch64-linux |
+| System        | URL                    |
+| :------------ | :--------------------- |
+| x86 Linux     | [Stable][iso-x86]      |
+| aarch64 Linux | [Stable][iso-aarch64]  |

Add these reference links near the other reference defs:

[iso-x86]: https://install.determinate.systems/nixos-iso/stable/x86_64-linux
[iso-aarch64]: https://install.determinate.systems/nixos-iso/stable/aarch64-linux