Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

docs: complete macOS node-gyp troubleshooting steps in CONTRIBUTING.md #4036

Merged
merged 7 commits into from
Jun 21, 2023

Conversation

OnlyOneJMJQ
Copy link
Contributor

Does what it says on the tin.

@davidmurdoch davidmurdoch changed the title Fill MacOS node-gyp troubleshooting guide docs: complete MacOS node-gyp troubleshooting steps Dec 15, 2022
@davidmurdoch davidmurdoch changed the title docs: complete MacOS node-gyp troubleshooting steps docs: complete macOS node-gyp troubleshooting steps in CONTRIBUTING.md Dec 15, 2022
CONTRIBUTING.md Outdated
@@ -31,7 +31,14 @@ If installation fails due to a `node-gyp` issue you may need to perform some add

### on macOS

- I have no idea.
- Determine if you have Xcode command line tools installed
- example the console will tell you if they're already installed): `xcode-select --install`
Copy link
Member

Choose a reason for hiding this comment

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

Can you make the steps more prescriptive? i.e., tell the user what to do directly.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This style (minus the missing "(") matches the other instructions. Would you like me to modify those too? They're short so it wouldn't take long.

CONTRIBUTING.md Outdated
- example: `xcode-select --install`
- Determine if you have Python 2.x installed
- example: `which python`
- If you do not have Python 2.x installed, you need to install it. We recommend `pyenv` for this so you can keep the default MacOS python installation untouched.
Copy link
Member

Choose a reason for hiding this comment

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

I don't know Python things... but does this need to be Python 2.7.x?

Copy link
Contributor

Choose a reason for hiding this comment

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

which python won't necessarily tell you which version is installed (for example, if you use pyenv, it'll just show you the shim).

python --version is probably more useful.

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated
- example: `xcode-select --install`
- Determine if you have Python 2.x installed
- example: `which python`
- If you do not have Python 2.x installed, you need to install it. We recommend `pyenv` for this so you can keep the default MacOS python installation untouched.
Copy link
Contributor

Choose a reason for hiding this comment

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

which python won't necessarily tell you which version is installed (for example, if you use pyenv, it'll just show you the shim).

python --version is probably more useful.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jun 21, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: ac8af96
Status: ✅  Deploy successful!
Preview URL: https://36222ead.ganache.pages.dev
Branch Preview URL: https://ngyp-macos-troubleshooting.ganache.pages.dev

View logs

CONTRIBUTING.md Outdated Show resolved Hide resolved
jeffsmale90 and others added 2 commits June 22, 2023 06:13
Co-authored-by: David Murdoch <187813+davidmurdoch@users.noreply.github.com>
@jeffsmale90 jeffsmale90 merged commit ac5deea into develop Jun 21, 2023
@jeffsmale90 jeffsmale90 deleted the ngyp-macos-troubleshooting branch June 21, 2023 20:30
@gitpoap-bot
Copy link

gitpoap-bot bot commented Jun 21, 2023

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2023 Ganache Contributor:

GitPOAP: 2023 Ganache Contributor GitPOAP Badge

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.

@davidmurdoch
Copy link
Member

image

@MicaiahReid <3

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants