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 LICENSE #2887

Merged
merged 1 commit into from
Oct 21, 2017
Merged

Update LICENSE #2887

merged 1 commit into from
Oct 21, 2017

Conversation

sDaniel
Copy link
Contributor

@sDaniel sDaniel commented Oct 16, 2017

Copyright 2016 Daniel Cohen Gindi & Philipp Jahoda < replaced template with content

Copyright 2016 Daniel Cohen Gindi & Philipp Jahoda < replaced template with content
@codecov-io
Copy link

Codecov Report

Merging #2887 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2887   +/-   ##
=======================================
  Coverage   19.64%   19.64%           
=======================================
  Files         113      113           
  Lines       13792    13792           
=======================================
  Hits         2709     2709           
  Misses      11083    11083

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8245f32...d7cd7b9. Read the comment docs.

@@ -186,7 +186,7 @@ Apache License
same "printed page" as the copyright notice for easier
identification within third-party archives.

Copyright {yyyy} {name of copyright owner}
Copyright 2016 Daniel Cohen Gindi & Philipp Jahoda
Copy link
Member

@liuxuan30 liuxuan30 Oct 17, 2017

Choose a reason for hiding this comment

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

@danielgindi is this intended to use a template or forgot to change?
BTW, should it be 2017 or just keep the first year? like 2014 or 2015

Copy link
Collaborator

Choose a reason for hiding this comment

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

It was missed. We can update the year later

@pmairoldi pmairoldi merged commit bdbdaa6 into ChartsOrg:master Oct 21, 2017
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.

4 participants