search

metricfu / metric_fu

A fist full of code metrics
http://metricfu.github.com/metric_fu
MIT License
624 stars 96 forks source link

Feature/windows ci #285

Closed bergholdt closed 7 years ago

bergholdt commented 7 years ago

Setup CI pipeline on AppVeyor and fixed specs.

Fix #284

bergholdt commented 7 years ago

AppVeyor builds OK on my fork, but fails on main repo. It does not see PR or branch feature/windows-ci.

https://ci.appveyor.com/project/bergholdt/metric-fu https://ci.appveyor.com/project/bergholdt/metric-fu-cfbfx

Perhaps CI project need to be create by owner of organization

bf4 commented 7 years ago

@bergholdt do you need help setting this up?

bergholdt commented 7 years ago

@bf4 yes I need help. I'm not able to trigger builds on this repo, only my own fork https://ci.appveyor.com/project/bergholdt/metric-fu/build/1.0.4

bf4 commented 7 years ago

I think I just set this up with access for you.

jkeam commented 7 years ago

Need any help getting this PR ready to merge?

bergholdt commented 7 years ago

All build passed 😊

Will remove bundle env, it's properly to verbose...extra commit will also show how much bundle caching helps

bergholdt commented 7 years ago

cache did not have any effect - must be using it wrong :-(

I think it's ready for merge - will you do it @jkeam