Skip to content

update LICENSE - fix year #49

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 28, 2017
Merged

update LICENSE - fix year #49

merged 1 commit into from
Oct 28, 2017

Conversation

@pushkyn
Copy link
Contributor

@pushkyn pushkyn commented Oct 28, 2017

Description of the change:
Reason for the change:
Link to original source:

Closes #46

Checklist

Make sure all of these items are complete, or else the PR will be ineligible for a code review.

  • Code passes all existing tests
  • Any new functionality added includes new unit tests
  • Create or update example code to show the new functionality in action.
  • All code, branch, and git naming and style conventions are followed (see CONTRIBUTING.md)
  • Feature branch has been rebased off of the latest master branch. ( see CONTRIBUTING.md).

@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label Oct 28, 2017
mbernier
mbernier suggested changes Oct 28, 2017
View reviewed changes
LICENSE.txt Outdated
The MIT License (MIT)

Copyright (c) 2016 SendGrid, Inc.
Copyright (c) 2017 SendGrid, Inc.
Copy link

Choose a reason for hiding this comment

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

Can you please update the year to be a range? This is my fault for assuming that would already have been there... Should be:

Copyright (c) 2016-2017 SendGrid, Inc.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@mbernier mbernier added difficulty: easy fix is easy in difficulty hacktoberfest labels Oct 28, 2017
@mbernier mbernier merged commit 75907f7 into sendgrid:master Oct 28, 2017
@thinkingserious
Copy link
Contributor

Hello @pushkyn,

Thanks again for the PR!

It's HACKTOBERFEST! We want to show our appreciation by sending you some special Hacktoberfest swag. If you have not already, could you please fill out this form so we can send it to you? Thanks!

Team SendGrid DX

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@mbernier mbernier mbernier approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

difficulty: easy fix is easy in difficulty status: code review request requesting a community code review or review from Twilio

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update the license file to have the correct date range

3 participants

@pushkyn @thinkingserious @mbernier