dendrograms / astrodendro

Generate a dendrogram from a dataset
https://dendrograms.readthedocs.io/
Other
37 stars 38 forks source link

Consolidate licenses #49

Closed ChrisBeaumont closed 11 years ago

ChrisBeaumont commented 11 years ago

Is there a reason why the license text is embedded in most source files? Can we consolidate that into a top level license file, or is that illegal somehow?

astrofrog commented 11 years ago

We can do like Astropy, and just put a LICENSE.rst file and refer to it at the top of the source code in a single line. I think that's usually assumed to be ok.

ChrisBeaumont commented 11 years ago

+1 for this (there's already a top level license file)

astrofrog commented 11 years ago

My favorite pull requests are ones that remove more than they add :)

ChrisBeaumont commented 11 years ago

Looks good (how did you turn an issue into a pull request??)

astrofrog commented 11 years ago

I used hub - https://github.com/defunkt/hub - for this one I did:

hub pull-request -i 49 -b dendrograms:master -h astrofrog:consolidate-license

Very handy!