buddhi-desilva/spree-pp-website-standard — comment on commit ba1aad0856285085d2c1013159171ec2257f08d0
File:  line 0
By tomash:

looks like a good job!

a few issues though:
- changing permissions of static files (paypal button image) from 644 to 755 is a bad idea (it's not an executable); are you developing on windows?
- I like the idea of moving test-suite gems from Gemfile to development dependency; does the test suite work after that change? i recall having a few issues with that approach some time ago.
- whitespace-changing lines are just noise in the commit (again: change system or configure editor)
- let's support ruby 1.9.2 and up, 1.9.1 is highly unstable and not recommended