Skip to content
This repository was archived by the owner on Apr 11, 2021. It is now read-only.

Add prerequistes to README#74

Open
smartcontracts wants to merge 1 commit intomasterfrom
smartcontracts-patch-1
Open

Add prerequistes to README#74
smartcontracts wants to merge 1 commit intomasterfrom
smartcontracts-patch-1

Conversation

@smartcontracts
Copy link
Copy Markdown

Description
Adds all necessary prerequisite software to the README. In response to issues that users are having when installing.

Metadata

Copy link
Copy Markdown
Contributor

@platocrat platocrat left a comment

Choose a reason for hiding this comment

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

looking good

Please make sure you've installed the following before continuing:

- [Git](https://git-scm.com/book/en/v2/Getting-Started-Installing-Git)
- [Node.js](https://nodejs.org/en/download/)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could add a .nvmrc and node to package.json


- [Git](https://git-scm.com/book/en/v2/Getting-Started-Installing-Git)
- [Node.js](https://nodejs.org/en/download/)
- [Yarn](https://classic.yarnpkg.com/en/docs/install#mac-stable)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could add .yarnrc also

- [Node.js](https://nodejs.org/en/download/)
- [Yarn](https://classic.yarnpkg.com/en/docs/install#mac-stable)
- `build-essential`
- `libusb-dev`
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

libusb is optional

- [Git](https://git-scm.com/book/en/v2/Getting-Started-Installing-Git)
- [Node.js](https://nodejs.org/en/download/)
- [Yarn](https://classic.yarnpkg.com/en/docs/install#mac-stable)
- `build-essential`
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These are platform specific so they should be under a header that says Ubuntu
build-essential is required for a C/C++ compiler so that node-gyp can use it

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

yarn install pre-requisite

4 participants