Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

doc: fix broken markdown link #319

Merged
merged 1 commit into from
Jun 28, 2017
Merged

doc: fix broken markdown link #319

merged 1 commit into from
Jun 28, 2017

Conversation

kfarnung
Copy link
Contributor

Checklist
Affected core subsystem(s)

doc

@kfarnung
Copy link
Contributor Author

kfarnung commented Jun 28, 2017

@kunalspathak It seems like we could simplify the build instructions a bit?

Build Command:

vcbuild [x86|x64|arm]

To run unit tests

vcbuild nobuild test ignore-flaky [x86|x64|arm]

EDIT: I've updated the build/test instructions to reflect the current state of the repo.

Copy link
Member

@kunalspathak kunalspathak left a comment

Choose a reason for hiding this comment

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

While you are at it, can you also include instructions to build test build and building with ninja?

@kfarnung
Copy link
Contributor Author

@kunalspathak and I chatted and he opened #320 to track advanced build scenarios.

* Fixed malformed link
* Added link to Homebrew
* Updated the build instructions

PR-URL: nodejs#319
Reviewed-By: Kunal Pathak <kunal.pathak@microsoft.com>
@kfarnung kfarnung merged commit 21ac436 into nodejs:master Jun 28, 2017
@kfarnung kfarnung deleted the readme branch June 28, 2017 22:23
kfarnung added a commit to kfarnung/node-chakracore that referenced this pull request Jul 7, 2017
* Fixed malformed link
* Added link to Homebrew
* Updated the build instructions

PR-URL: nodejs#319
Reviewed-By: Kunal Pathak <kunal.pathak@microsoft.com>
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.

2 participants