LICENSE? #36

Closed
Niggler opened this Issue Jul 31, 2013 · 4 comments

Comments

Projects
None yet
3 participants
@Niggler

Niggler commented Jul 31, 2013

You know the drill

@JamieMason

This comment has been minimized.

Show comment
Hide comment
@JamieMason

JamieMason Jul 31, 2013

Owner

Hey, currently the license is in the source of the shell and applescript. Please add some more detail and I'll see what I can do, thanks.

Owner

JamieMason commented Jul 31, 2013

Hey, currently the license is in the source of the shell and applescript. Please add some more detail and I'll see what I can do, thanks.

@jamesstout

This comment has been minimized.

Show comment
Hide comment
@jamesstout

jamesstout Jul 31, 2013

Contributor

Normally you just whack a LICENSE file in the root of the repo. For example: https://github.com/jamesstout/AppleScripts/blob/master/LICENSE

Contributor

jamesstout commented Jul 31, 2013

Normally you just whack a LICENSE file in the root of the repo. For example: https://github.com/jamesstout/AppleScripts/blob/master/LICENSE

@Niggler

This comment has been minimized.

Show comment
Hide comment
@Niggler

Niggler Jul 31, 2013

@jamesstout the attribution in this case is perfectly fine

@JamieMason you should add the relevant line to package.json, as described in the docs: https://npmjs.org/doc/files/package.json.html#license

Niggler commented Jul 31, 2013

@jamesstout the attribution in this case is perfectly fine

@JamieMason you should add the relevant line to package.json, as described in the docs: https://npmjs.org/doc/files/package.json.html#license

@JamieMason JamieMason closed this in f082a3e Aug 1, 2013

@JamieMason

This comment has been minimized.

Show comment
Hide comment
@JamieMason

JamieMason Aug 1, 2013

Owner

thanks @Niggler, I've done both - appreciate that.

Owner

JamieMason commented Aug 1, 2013

thanks @Niggler, I've done both - appreciate that.

JamieMason added a commit that referenced this issue Aug 4, 2013

Merge branch 'master' of github.com:JamieMason/ImageOptim-CLI
* 'master' of github.com:JamieMason/ImageOptim-CLI:
  LICENSE, fixes #36

JamieMason added a commit that referenced this issue Aug 7, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment