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

Add xarray copyright notice to comply with reuse under Apache License #35213

Merged
merged 1 commit into from
Jul 10, 2020

Conversation

jthielen
Copy link
Contributor

@jthielen jthielen commented Jul 10, 2020

In xarray-contrib/pint-xarray#11, @keewis and I noticed that pandas's (pandas'?...possessive forms are weird) reuse with modification of xarray code added in eee83e2 did not include xarray's copyright notice as required by the Apache License, Version 2.0 (which is not included in the license text itself, unlike MIT or BSD). I'm not sure how big of a deal this is, but given that pint-xarray is going ahead and modeling its code citation practice after that of pandas, I wanted to see what the thoughts are here on it. This PR takes a simple fix and just adds the copyright notice to top of the LICENSES/XARRAY_LICENSE file.

Also, I'm not sure if this requires a whatsnew entry or not.

  • closes #xxxx
  • tests added / passed
  • passes black pandas
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • whatsnew entry

@jreback jreback added the Docs label Jul 10, 2020
@jreback jreback added this to the 1.1 milestone Jul 10, 2020
@jreback
Copy link
Contributor

jreback commented Jul 10, 2020

sure

@jreback jreback merged commit 280efbf into pandas-dev:master Jul 10, 2020
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.

2 participants