Skip to content

Conversation

oesteban
Copy link
Member

This PR addresses the discussion in nipreps/fmriprep#2325.

@satra
Copy link

satra commented Apr 29, 2021

LGTM. thanks for clarifying all of this.

@effigies
Copy link
Member

Note from @satra:

also while header per file could useful, not necessary: https://infra.apache.org/apply-license.html#copy-per-file

@oesteban
Copy link
Member Author

As per our conversation in the roundup today, this would be the first leg of the license change. If no opposing voices comment here before next Thursday, then this will be merged on Friday. Please note that merging this document does not mean we have "passed" the relicensing of fMRIPrep (that's the other leg).
/cc @nipreps/fmriprep-contributors .

Re: missing header - current text already considers that (and recommends third-parties to add the header if it were missing, as the Apache license already recommends).

@oesteban
Copy link
Member Author

Since this is not a license change itself and no voices have objected this far, I think it is pretty safe to pull it in. Thank you all for participating!

@oesteban oesteban merged commit 0375f5d into mkdocs May 14, 2021
@oesteban oesteban deleted the enh/licensing-apache branch May 14, 2021 07:07
@oesteban
Copy link
Member Author

The new documents are up: https://www.nipreps.org/community/licensing/

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.

3 participants