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

Added yarn install to contribute instructions #2117

Merged
merged 2 commits into from
Sep 3, 2017
Merged

Added yarn install to contribute instructions #2117

merged 2 commits into from
Sep 3, 2017

Conversation

vallamost
Copy link
Contributor

The contribute instructions don't mention installing yarn ahead of time in case the builder doesn't already have it installed.

Ian added 2 commits August 24, 2017 20:47
The contribute instructions don't mention installing yarn ahead of time in case the builder doesn't already have it installed.
The yarn MSI installer requires npm as mentioned on the install instructions
Copy link
Contributor Author

@vallamost vallamost left a comment

Choose a reason for hiding this comment

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

Added note about needing npm

Copy link
Contributor

@albinekb albinekb left a comment

Choose a reason for hiding this comment

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

Cool! thanks 😃

@chabou chabou merged commit 537e58e into vercel:master Sep 3, 2017
@chabou
Copy link
Collaborator

chabou commented Sep 3, 2017

Thank you for your help 🙏

chabou added a commit that referenced this pull request Sep 3, 2017
* master: (62 commits)
  1.4.3
  Disable ia32 linux releases (#2164)
  Fixed writing composed characters (#2158)
  Doc: Add yarn install to contribute instructions (#2117)
  Change "Close Session" shortcut on Linux/Windows  (#2160)
  Notice for plugins (#2114)
  Updated dependencies to the latest version (#2146)
  1.4.2
  Reverted class names to as they were before (#2139)
  1.4.1
  AppVeyor environment variables are now on the platform (#2137)
  Brought back the icon for closing tabs (#2136)
  Brought back keymap documentation to the website (#2133)
  1.4.0
  Don't build on master, except for releases (#2132)
  Ensured that `async-retry` is added to the bundle (#2131)
  Ensure correct update channel is displayed in About window (#2130)
  Retry loading it if config doesn't exist in auto updater (#2129)
  Write contents of default config to hyper.js (#2128)
  Use a string for setting the update channel (#2127)
  ...
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.

None yet

3 participants