Skip to content

Conversation

@mkamranhamid
Copy link
Contributor

@mkamranhamid mkamranhamid commented Oct 5, 2019

…yarn

What kind of change does this PR introduce?

Added a line in the doc contributing.md to have the latest version of yarn. I had yarn version 1.13 and our app required 1.19. When I did yarn install It didn't work so I had to check and install the latest version of yarn and it fixed the error.

What is the current behavior?

Currently, if you follow the instruction as I did and try yarn install you will probably end up in an error

What is the new behavior?

Now if you're following instruction and you see Its a requirement to have Node version 10.x.x. You will also see a point to have yarn latest version

What steps did you take to test this?

N/A

Checklist

  • Documentation
  • Testing
  • Ready to be merged
  • Added myself to contributors table

@vercel vercel bot temporarily deployed to staging October 5, 2019 17:43 Inactive
@vercel
Copy link

vercel bot commented Oct 5, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://codesandbox-clien-git-fork-mkamranhamid-doc-update-contr-5c1366.codesandbox1.now.sh

Copy link
Contributor

@SaraVieira SaraVieira left a comment

Choose a reason for hiding this comment

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

Good catch! Thank you

@SaraVieira SaraVieira merged commit 0453db5 into codesandbox:master Oct 5, 2019
@mkamranhamid
Copy link
Contributor Author

Thank you so much this is my first open source contribution

@mkamranhamid
Copy link
Contributor Author

Can you please add #hacktoberfest in the label. @SaraVieira

CompuIves pushed a commit that referenced this pull request Oct 17, 2019
…ce/Dependencies/AddVersion/index.js (#2671)

* add an information in contributing doc to have the latest version of yarn (#2662)

* Update index.js

* Update index.js

* Update and rename index.js to index.tsx
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.

2 participants