Skip to content
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

chore: update links #4225

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Rotzbua
Copy link
Contributor

@Rotzbua Rotzbua commented May 15, 2024

Change-type: patch

change to https if available
some urls changed

@@ -43,10 +43,10 @@ Etcher requires an available [polkit authentication agent](https://wiki.archlinu

## May I run Etcher in older macOS versions?

Etcher GUI is based on the [Electron](http://electron.atom.io/) framework, [which only supports macOS 10.10 and newer versions](https://github.com/electron/electron/blob/master/docs/tutorial/support.md#supported-platforms).
Etcher GUI is based on the [Electron](https://www.electronjs.org/) framework, [which only supports macOS 10.10 and newer versions](https://github.com/electron/electron/blob/main/README.md#platform-support).
Copy link
Contributor

Choose a reason for hiding this comment

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

That link says "macOS (Catalina and up)" and https://en.wikipedia.org/wiki/MacOS_Catalina says that this is macOS 10.15 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just updated the link and did not verify every text. When the faq was written the macOS 10.10 was the current state. The previous link points to the master-branch so it is not clear for which electron it was the case. Maybe remove the explicit macOS version and just keep the reference to electron?

docs/USER-DOCUMENTATION.md Outdated Show resolved Hide resolved
Change-type: patch

change to https if available
some urls changed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants