Skip to content

Conversation

@lagru
Copy link
Member

@lagru lagru commented Jun 19, 2024

Closes #3 because I think all other things are addressed.

Is it okay to grant the additional copyright like I did here and keep the old one? In this case I'd be totally happy to remove the line with my name, but I'd find it a bit strange if we required tools with a longer history that move over to the Scientific Python umbrella to forsake their copyright (I think in some jurisdictions that isn't even possible)? If we require this, the docs definitely need some documentation around this expectation, as I'm myself a bit unclear on what this means, why it's necessary, and its consequences.

@lagru lagru requested review from bsipocz and jarrodmillman June 19, 2024 12:17
Copy link
Member

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

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

This is exactly what we did with doctestplus, but swapping the order, the newest/actual copyright is at the top, then history at the bottom. This one might look a bit weird as all the years are the same, but time will sort it all out on the long term :)

(code suggestion didn't work, so here it is:

Copyright (c) 2024, Scientific Python Developers
Copyright (c) 2024, Lars Grüter

@jarrodmillman
Copy link
Member

You should leave your name! Just switch the order and this is ready to merge.

as well instead of just "docstub developers".
@lagru lagru force-pushed the add-sp-copyright branch from 874de77 to c78285d Compare June 21, 2024 07:18
@lagru lagru merged commit b8c453f into main Jun 21, 2024
@lagru lagru deleted the add-sp-copyright branch June 21, 2024 07:19
@lagru
Copy link
Member Author

lagru commented Jun 21, 2024

Thanks for the feedback. :D

@lagru lagru added the documentation Improvements or adds to documentation label Sep 19, 2024
@lagru lagru restored the add-sp-copyright branch May 25, 2025 13:58
@lagru lagru deleted the add-sp-copyright branch May 25, 2025 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or adds to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review tools policy

4 participants