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

Update README.md #3

Closed
wants to merge 2 commits into from
Closed

Update README.md #3

wants to merge 2 commits into from

Conversation

duncup
Copy link

@duncup duncup commented Apr 15, 2019

The pull request will be closed without any reasons if it does not satisfy any of following requirements:

  1. Make sure you are targeting the master branch, pull requests on release branches are only allowed for bug fixes.
  2. Please read contributing guidelines: CONTRIBUTING
  3. Describe what your pull request does and which issue you're targeting (if any and Please use English)
  4. ... if it is not related to any particular issues, explain why we should not reject your pull request.
  5. The Commits must use English, must be test and No useless submissions.

You MUST delete the content above including this line before posting, otherwise your pull request will be invalid.

Please provide Issues links to:

Provide test code:

    

Description

...

@duncup duncup requested a review from vcaesar as a code owner April 15, 2019 09:54
@vcaesar vcaesar closed this in 3deaca0 Apr 15, 2019
@vcaesar
Copy link
Owner

vcaesar commented Apr 15, 2019

In fact, if node installed go and gcc, you only need to execute npm install robotn. You do not need to download other tools and build from source code.

@vcaesar vcaesar added the question Further information is requested label Apr 15, 2019
@vcaesar
Copy link
Owner

vcaesar commented Apr 15, 2019

Thanks for your PR, I updated README.md, deleted the unclear statement; Because package.json performs these operations, I am considering whether to add go get -v github.com/go-vgo/robotgo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants