Oops - misread the docs. We've used a `-` as a bullet item marker in the return code docs so I accidentally read `1` with a `-` bullet marker in front of it as `-1`. Nonetheless, I think the doc suggests that we should be returning 0 if refresh fails.