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

Standardized Readme #2883

Merged
merged 1 commit into from
Jun 21, 2016
Merged

Standardized Readme #2883

merged 1 commit into from
Jun 21, 2016

Conversation

RichardLitt
Copy link
Member

@RichardLitt RichardLitt added the need/review Needs a review label Jun 21, 2016
See ipfs/community#124

License: MIT
Signed-off-by: Richard Littauer <richard.littauer@gmail.com>
@whyrusleeping
Copy link
Member

LGTM

@RichardLitt RichardLitt added RFM and removed need/review Needs a review labels Jun 21, 2016
@whyrusleeping whyrusleeping merged commit 30ef23a into master Jun 21, 2016
@whyrusleeping whyrusleeping deleted the feature/standardize-readme branch June 21, 2016 22:25
@@ -1,6 +1,6 @@
The MIT License (MIT)

Copyright (c) 2014 Juan Batiz-Benet
Copyright (c) 2016 Juan Batiz-Benet
Copy link
Member

Choose a reason for hiding this comment

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

No, you don't just "update" a copyright notice. the whole point is the earliest time. Also, in general don't remove things from copyright notices. should usually only add.

So if you want to say 2016, then you'd have to say:

Copyright (c) 2014 Juan Batiz-Benet
Copyright (c) 2016 Juan Batiz-Benet

or

Copyright (c) 2014-2016 Juan Batiz-Benet

or

Copyright (c) since 2014 Juan Batiz-Benet

Copy link
Member Author

Choose a reason for hiding this comment

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

SGTM. Thanks.

RichardLitt added a commit that referenced this pull request Aug 29, 2016
See #2883 (comment)

License: MIT
Signed-off-by: Richard Littauer <richard.littauer@gmail.com>
@RichardLitt RichardLitt mentioned this pull request Aug 29, 2016
RichardLitt added a commit that referenced this pull request Aug 29, 2016
See #2883 (comment). Will need to change current issues.

License: MIT
Signed-off-by: Richard Littauer <richard.littauer@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants