wk8 / cookbook-cloudflare

A Chef cookbook to register your servers with Cloudflare's DNS & threat control services
The Unlicense
7 stars 12 forks source link

Enhance quality of this cookbook #11

Open rmoriz opened 9 years ago

rmoriz commented 9 years ago

Thanks for the cookbook. Please consider the following enhancements:

Thanks!

wk8 commented 9 years ago

Hi @rmoriz ,

Thanks for the feedback!

I'm fully aware of this cookbook's poor quality, as it happens, and I've been wanting to add TK tests for a looong time, but never had the time to do so. Just wrote it to do the job when I needed it, but never had the time to bring it up to par with usual standards.

If you use this cookbook, maybe you'll want to open a few PRs to address some of the issues you highlight above? :-)

rmoriz commented 9 years ago

@wk8 I can do that, but the current licensing is a show-stopper for me.

wk8 commented 9 years ago

If you make a PR to change it to MIT (bumping the version, since the current version's license can't be modified), I'll merge it.

That being said, why is the licensing such a big deal to you, if you don't mind me asking?

rmoriz commented 9 years ago

@wk8 Isn't OpenSource about an open license? When I contribute to a project, I want to be able to use my contribution. I trust in OSI's analysis of various licenses. Almost all chef cookbooks are Apache licensed, most Ruby gems are MIT-licensed. Both do the job very good IMHO.

wk8 commented 9 years ago

@rmoriz sure is; but seemed to me the current licensing's even more permissive, so I was wondering what's so wrong with it in your opinion.

In any case, happy to change to MIT, as stated above.