-
Notifications
You must be signed in to change notification settings - Fork 81
Add note for older macOS Nix installation #489
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Added a note about using the Universal graphical installer for older macOS versions, because the curl CLI fails at least on some Mac, because it tries to fetch https://install.determinate.systems/nix/tag/v3.13.2/nix-installer-x86_64-darwin which does not exist.
✅ Deploy Preview for zero-to-nix ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
WalkthroughAdds fallback guidance to documentation for users on older macOS versions. If the installer command fails, users are directed to the Universal graphical installer. No code logic or behavior is modified. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Tip 📝 Customizable high-level summaries are now available in beta!You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.
Example instruction:
Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later. 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. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/content/concepts/nix-installer.mdx (1)
33-33: Verify the fallback URL and convert to a markdown link.The note directs users to a general docs URL without a specific path. Verify that
https://docs.determinate.systemsis reachable and clearly documents the Universal graphical installer. Additionally, convert the URL to a markdown link for better user experience.-If this does not work (e.g. on older macOS versions), then go to https://docs.determinate.systems and use e.g. the _Universal_ graphical installer. +If this does not work (e.g. on older macOS versions), then go to [https://docs.determinate.systems](https://docs.determinate.systems) and use the _Universal_ graphical installer.Note: I removed the second "e.g." since the Universal installer appears to be the specific recommendation rather than one of several options.
Added a note about using the Universal graphical installer for older macOS versions, because the curl CLI fails at least on some Mac, because it tries to fetch https://install.determinate.systems/nix/tag/v3.13.2/nix-installer-x86_64-darwin which does not exist.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.