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

Added a top level license file #7

Merged
merged 1 commit into from Nov 18, 2014
Merged

Added a top level license file #7

merged 1 commit into from Nov 18, 2014

Conversation

pmitros
Copy link
Contributor

@pmitros pmitros commented Nov 18, 2014

The course is offered under CC-BY, which is fantastic. This is not obvious from the repository (unless you dig into the iPython notebooks). A top-level license file makes this obvious both to people browsing and to search engines.

@gforsyth
Copy link
Member

Hey @pmitros -- yes, this definitely needs to be done. All the code, however, needs to be listed under an MIT license. Do you think we can put both agreements in the same LICENSE.md?

@ahmadia
Copy link
Collaborator

ahmadia commented Nov 18, 2014

@gforsyth - yeah that's fine.

@ahmadia
Copy link
Collaborator

ahmadia commented Nov 18, 2014

The correct thing to do is put the statement from the notebooks at the top, then copy the full licenses below.

@ahmadia
Copy link
Collaborator

ahmadia commented Nov 18, 2014

This is a pretty good starting point: https://github.com/swcarpentry/bc/blob/gh-pages/LICENSE.md

@pmitros
Copy link
Contributor Author

pmitros commented Nov 18, 2014

Rebased. How about this?

I kept this as a plain text file. github defaults to plain text for the LICENSE files, and I followed their convention for maximum compatibility with search engines.

@ahmadia
Copy link
Collaborator

ahmadia commented Nov 18, 2014

Thanks, @gforsyth I'm going to merge this.

ahmadia added a commit that referenced this pull request Nov 18, 2014
Added a top level license file
@ahmadia ahmadia merged commit c91b54e into barbagroup:master Nov 18, 2014
@gforsyth
Copy link
Member

Needs editing -- still refers to Software Carpentry in the license -- I'll edit it on master

@gforsyth
Copy link
Member

also, thanks again @pmitros !

@ahmadia
Copy link
Collaborator

ahmadia commented Nov 18, 2014

@gforsyth - my mistake, I didn't even catch that. I'll fix.

@gforsyth
Copy link
Member

@ahmadia I got it already, no worries

@pmitros
Copy link
Contributor Author

pmitros commented Nov 18, 2014

Sorry. My bad.

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.

None yet

3 participants