Closed hweng closed 8 years ago
👍 looks good to me, but will let @lsitu and @VivianChu do the formal review
👍
@hweng I see lots of changes on Gemfile.lock file but can't find anything committed to Gemfile. Do you make any changes to the Gemfile?
@lsitu No, there is no new gem added in this pull request. Have you pulled the latest version from github master branch and run bundle install and do you get error? I got errors which I have to delete the Gemfile.lock and then run bundle install and it get fixed. That's why the Gemfile.lock is changed.
@hweng It looks like some weird environment but I think it's fine as long as it works. Actually I didn't get any errors with my last PR that is basing on the latest version from github master branch.
Thanks everyone for reviewing and @lsitu for merging.
@ucsdlib/developers Please review, comments etc... Thanks.