Skip to content
This repository has been archived by the owner. It is now read-only.

Add license file respecting history from imported content #36

Open
wants to merge 1 commit into
base: master
from

Conversation

@RealOrangeOne
Copy link
Member

@RealOrangeOne RealOrangeOne commented Nov 25, 2017

This is MIT with additional contributors, due to importing kit and
board docs and whirlwind tour from https://github.com/srobo/docs.

This is MIT with additional contributors, due to importing kit and
board docs and whirlwind tour from https://github.com/srobo/docs.
@RealOrangeOne RealOrangeOne requested review from PeterJCLaw and Adimote Nov 25, 2017
@@ -0,0 +1,29 @@
Copyright (c) 2010-2017 Chris Kirkham <[email protected]>
Richard Barlow <[email protected]>

This comment has been minimized.

@PeterJCLaw

PeterJCLaw Nov 26, 2017
Contributor

It'd be good if we could have consistent whitespace here (I suggest using spaces since we want to align with the top line).

@PeterJCLaw
Copy link
Contributor

@PeterJCLaw PeterJCLaw commented Nov 26, 2017

@RealOrangeOne I've been thinking about this some more and I'm increasingly of the opinion that we should be clear about the license of this work as separate from the bits pulled in from elsewhere. I think we maybe want to go with something that clarifies the different contributions.

I've also run a blame on srweb's docs (I think we're interested in the slightly more restricted git shortlog -sne -- content/en/docs/kit/ content/en/docs/tutorials/python | cut -d ' ' -f 2), which yields quite a few more contributors (I'll be upstreaming the wider additions to the copyright holders for the srobo/docs repo shortly).

@PeterJCLaw
Copy link
Contributor

@PeterJCLaw PeterJCLaw commented Feb 6, 2018

@RealOrangeOne why was this frozen (and what does that mean)?

@RealOrangeOne
Copy link
Member Author

@RealOrangeOne RealOrangeOne commented Feb 6, 2018

it was frozen as a note for me not to merge at kickstart. License things came up in the last meeting, and are being researched into, apparently

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.