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 copyright year, and delete duplicate license file. #1706

Merged
merged 1 commit into from Sep 8, 2017

Conversation

Projects
None yet
1 participant
@jasongrout
Member

jasongrout commented Sep 8, 2017

Fixes #1701

@jasongrout

This comment has been minimized.

Member

jasongrout commented Sep 8, 2017

I'm choosing to keep the LICENSE file as-is and delete the COPYING.md based on the project discussions at https://groups.google.com/forum/#!topic/jupyter/vZpWgw8zKdc

@jasongrout jasongrout merged commit 6e26a66 into jupyter-widgets:master Sep 8, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jasongrout

This comment has been minimized.

Member

jasongrout commented Sep 8, 2017

I reverted this in a force-push to master, since it should not have been merged. Instead, we should wait until jupyter/governance#37 is resolved and follow that project guidance.

@jasongrout jasongrout added this to the Reference milestone Oct 19, 2017

@jasongrout jasongrout referenced this pull request Apr 27, 2018

Closed

Package license files #2048

jasongrout added a commit to jasongrout/ipywidgets that referenced this pull request Jul 16, 2018

Clarify license files.
Based on discussion at jupyter/governance#37 and discussion from the links on that issue, I made the following changes:

1. Delete the COPYING.md, as it was redundant
2. Changed the LICENSE copyright dates to just have the starting year

I also added LICENSE to the widgetsnbextension package (thanks to @jakirkham for noticing this in jupyter-widgets#2048).

This supersedes jupyter-widgets#1706, which was reverted in master while the discussion was still happening.

@jasongrout jasongrout referenced this pull request Jul 16, 2018

Merged

Clarify license files. #2133

@jasongrout

This comment has been minimized.

Member

jasongrout commented Jul 17, 2018

Fixed (finally) by #2133.

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