Skip to content

Conversation

@techne98
Copy link

@techne98 techne98 commented Dec 22, 2025

This PR is in reference to #1010 and aims to create a consistent stylization of "macOS" in the install documentation.

Some pages and paragraphs use "MacOS", some use "macOS", and so this consolidates all references to "macOS", which is in line with Apple's current branding strategy.

It also makes some minor wording changes in the install/on_mac_os page to make the installation instructions feel a little tighter.

I hope this helps.

Jack

This updates some of the wording on the install page for macOS to
make it a little bit tighter and concise.
@netlify
Copy link

netlify bot commented Dec 22, 2025

Deploy Preview for crystal-website ready!

Name Link
🔨 Latest commit 5bea74b
🔍 Latest deploy log https://app.netlify.com/projects/crystal-website/deploys/69496ca8b77b240008cd8cc6
😎 Deploy Preview https://deploy-preview-1012--crystal-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@straight-shoota
Copy link
Member

We should add the configuration to _typos.toml in order to maintain the spelling in the future:

--- i/_typos.toml
+++ w/_typos.toml
@@ -5,6 +5,7 @@ contextful = "contextful"
 ba = "ba"
 Flate = "Flate"
 "Haß" = "Haß"
+MacOS = "macOS"
 wrk = "wrk"
 SEH = "SEH"
 Packt = "Packt

@techne98
Copy link
Author

techne98 commented Dec 22, 2025

We should add the configuration to _typos.toml in order to maintain the spelling in the future:

--- i/_typos.toml
+++ w/_typos.toml
@@ -5,6 +5,7 @@ contextful = "contextful"
 ba = "ba"
 Flate = "Flate"
 "Haß" = "Haß"
+MacOS = "macOS"
 wrk = "wrk"
 SEH = "SEH"
 Packt = "Packt

I have added this entry to the _typos.toml file as suggested.

I also deleted the _site directory locally and updated the /_data/nav.yml file and it seems to be working.

There are two install keys in the YAML file - I changed MacOS to macOS in both instances.

@techne98 techne98 marked this pull request as ready for review December 22, 2025 12:14
@straight-shoota
Copy link
Member

Oh that duplicate should not be there: #1015

@techne98
Copy link
Author

Ahh, I was wondering, but was unsure.

Is there anything you want me to do on my end, or will you merge #1015 and I can fix the conflicts here?

@techne98
Copy link
Author

I've updated the capitalization of macOS in the link_actions on the install.md page which should fix the error in the Lint workflow.

Not too sure about the build one failing

@straight-shoota
Copy link
Member

The build job failure is unrelated.

@techne98 techne98 closed this by deleting the head repository Jan 10, 2026
@straight-shoota
Copy link
Member

@techne98 did you delete your fork intentionally?

@techne98
Copy link
Author

Hello, I accidentally deleted the wrong repository 🥲

@straight-shoota
Copy link
Member

No problem.
If you restore the repo, we might be able to reopen this PR. Otherwise you can just create a new one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants