tjwp commented on pull request #1186:
URL: https://github.com/apache/avro/pull/1186#issuecomment-819763618


   @RyanSkraba I'd appreciate if you could take another look. Here are the 
changes I made:
   - Updated `BUILD.md`. I removed gems that will be installed by bundler in 
the build.sh script from the requirements.
   - Unpinned the gems in the Gemfile. I kept a version restriction on Rubocop 
for now because upgrading further requires additional code changes, and those 
should be addressed separately.
   - Updated the Dockerfile to install and use Ruby 2.6.7. This part of the 
change is an updated version of https://github.com/apache/avro/pull/929.
   
   If this looks good, then I can squash into one commit before merging.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to