StackStorm / chef-stackstorm

DEPRECATED! Community-maintained Chef Cookbook to deploy StackStorm, IFTTT for Ops
https://supermarket.chef.io/cookbooks/stackstorm
Apache License 2.0
16 stars 10 forks source link

Add FoodCritic and Rubocop testing #1

Closed shortdudey123 closed 8 years ago

arm4b commented 8 years ago

@shortdudey123 this is really nice, but these recipes (incl. https://github.com/StackStorm/chef-openstack-mistral) are really outdated and codify old installation way, which is not supported anymore.

Entire installation was reworked, where all pip requirements are already bundled in deb/rpm native packages.

You can use the following (fresh) instructions to install StackStorm: https://docs.stackstorm.com/install/index.html

shortdudey123 commented 8 years ago

@armab i am aware of that, one step at a time :) That part will be in another PR once this repo is a bit more standardized

arm4b commented 8 years ago

Awesome! :+1:

arm4b commented 8 years ago

And sorry for this confuse, because we should place some notice beforehand, indicating that repo is outdated/requires re-work after we released new packages.

shortdudey123 commented 8 years ago

np

shortdudey123 commented 8 years ago

Let me know you feedback on the updates here If you guys have travis turned on for this repo (or want to turn it on) then i can add a .travis.yml file as well

Kami commented 8 years ago

@shortdudey123 Yeah, please add .travis.yml file and I will enable Travis for this repo.

Thanks!

enykeev commented 8 years ago

I've enabled it. Now we only need a file.

shortdudey123 commented 8 years ago

@Kami @enykeev all done :)

dennybaa commented 8 years ago

@shortdudey123 Thank you! Looks good) @Kami @enykeev, I'm +1 to merge this.

Kami commented 8 years ago

Merged, thanks!