-
Notifications
You must be signed in to change notification settings - Fork 24
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/package.json files #101
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 💯
@@ -32,6 +32,7 @@ | |||
"webpack": "^4.5.0", | |||
"webpack-cli": "^2.0.14" | |||
}, | |||
"contributors": ["Christian Maniewski <chris@colony.io>", "James Lefrère <james@colony.io>"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be authors, as contributors are a lot more: https://github.com/JoinColony/colonyJS/graphs/contributors
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I just now read the PR's description:
Replaces author with contributors in all package.json files
Weird...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True! But then again, they didn't directly contribute to our repo 🙃
npm just references author
or contributors
, so I guess we can go with this? https://docs.npmjs.com/files/package.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But what happens when there are "real" contributors ? :)
29daac6
to
bd037f9
Compare
Description
This PR updates all README files so that they are written in the same style and link to the colonyJS docs site where appropriate.
Other changes (e.g. bug fixes, UI tweaks, refactors)
Readme.md
toREADME.md
(just so we have a standard)author
withcontributors
in allpackage.json
filescolony-js
on GithubContributes to #100