Use MIT as default cookbook license #119

raucao · 2019-12-19T11:51:31Z

raucao commented

2019-12-19 11:51:31 +00:00

And improve the author name, making it more consistent with the rest of the repo.

greg commented

2019-12-20 15:56:04 +00:00

I gave this a try, I know this used to be the way to set the defaults for the cookbook generate. The docs say that is still the case, but they are using default values when I use chef generate cookbook COOKBOOK_NAME. Figuring it out

I gave this a try, I know this used to be the way to set the defaults for the cookbook generate. [The docs](https://docs.chef.io/cookbook_repo.html) say that is still the case, but they are using default values when I use `chef generate cookbook COOKBOOK_NAME`. Figuring it out

greg commented

2019-12-27 10:41:31 +00:00

I found a way to make it work (d029ee91be), but it wasn't easy as this is undocumented and only works with a config.rb file at the root of the repo, not at the usual locations. I have asked in the issue about this being documented if I should create a new issue about it: https://github.com/chef/chef-cli/issues/54#issuecomment-569243112

I found a way to make it work (d029ee91be), but it wasn't easy as this is undocumented and only works with a `config.rb` file at the root of the repo, not at the usual locations. I have asked in the issue about this being documented if I should create a new issue about it: https://github.com/chef/chef-cli/issues/54#issuecomment-569243112

raucao commented

2019-12-27 11:06:18 +00:00

I only changed it so that the documented default isn't wrong. It's easy enough for anyone to type a line of text when they create a cookbook. Spending a ton of time on this is not really useful IMO.

This was meant to be looked at within like 5 seconds and be merged without kredits, when I opened it. I won't be testing any changes, so whatever you did, please manage it yourself and give yourself kredits if you want to.

I only changed it so that the documented default isn't wrong. It's easy enough for anyone to type a line of text when they create a cookbook. Spending a ton of time on this is not really useful IMO. This was meant to be looked at within like 5 seconds and be merged without kredits, when I opened it. I won't be testing any changes, so whatever you did, please manage it yourself and give yourself kredits if you want to.

greg closed this pull request

2019-12-27 15:32:14 +00:00

greg closed this pull request

2019-12-27 15:32:14 +00:00

greg deleted branch chore/default_license

2019-12-27 15:32:21 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kosmos/chef#119