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

Documentation: Clarify that Calypso does not run on Windows #2093

Merged
merged 1 commit into from
Jan 5, 2016

Conversation

nylen
Copy link
Contributor

@nylen nylen commented Jan 5, 2016

Suggested by @hughc in #2029 (comment).

Also cleaned up a couple of other things here:

  • Add link to brew for OS X
  • Remove misleading link pointing to Node.js binaries (it's very easy to get to a better download page from the Node.js homepage)

@nylen nylen added Documentation [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. OSS Citizen labels Jan 5, 2016
@nylen nylen self-assigned this Jan 5, 2016
@nylen nylen changed the title Clarify that Calypso does not run on Windows Documentation: Clarify that Calypso does not run on Windows Jan 5, 2016
- [Git](http://git-scm.com/). Try the `git` command from your terminal, if it's not found then use this [installer](http://git-scm.com/download/).
- The repository also uses `make` to orchestrate compiling the JavaScript, running the server, and several other tasks. On Mac OSX, the easiest way to install `make` is through Apple's [Command Line Tools for Xcode](https://developer.apple.com/downloads/) (requires free registration).
- The repository also uses `make` to orchestrate compiling the JavaScript, running the server, and several other tasks. On Mac OS X, the easiest way to install `make` is through Apple's [Command Line Tools for Xcode](https://developer.apple.com/downloads/) (requires free registration).
Copy link
Member

Choose a reason for hiding this comment

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

Nice fix - I just double-checked with Apple to make sure that this is the appropriate way to write Mac OS X and it looks like it's perfect ✅

@dmsnell
Copy link
Member

dmsnell commented Jan 5, 2016

LGTM

@dmsnell dmsnell added [Status] Ready to Merge and removed [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Jan 5, 2016
nylen added a commit that referenced this pull request Jan 5, 2016
Documentation: Clarify that Calypso does not run on Windows
@nylen nylen merged commit 9c4bf90 into master Jan 5, 2016
@nylen nylen deleted the docs/no-windows branch January 5, 2016 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants