Skip to content
This repository has been archived by the owner on Mar 24, 2020. It is now read-only.

Fixes #293 - Upgrade Nokogiri to address security vulnerability. #294

Merged
merged 2 commits into from
Mar 30, 2017

Conversation

lsitu
Copy link
Member

@lsitu lsitu commented Mar 29, 2017

Fixes #293 ; refs #293

Upgrade Nokogiri to the latest 1.7.1 to address security vulnerability.

Changes proposed in this pull request:

  • Upgrade Nokogiri to 1.7.1

@ucsdlib/developers - please review

@mcritchlow
Copy link
Member

@lsitu looks like the branch is out of date with develop (probably just @remerjohnson's latest changes) would you mind updating? sadly it will mean kicking off circle ci again :(

@lsitu
Copy link
Member Author

lsitu commented Mar 29, 2017

Yes, I realize that but hesitate to do it since the build is so slow. I'll do it now.

@coveralls
Copy link

Coverage Status

Coverage increased (+15.4%) to 66.54% when pulling 45f51ae on feature/nokogili_upgrade into 74e9b7c on develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+15.4%) to 66.54% when pulling 8e5fe76 on feature/nokogili_upgrade into 74e9b7c on develop.

Copy link
Member

@mcritchlow mcritchlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@hweng
Copy link
Contributor

hweng commented Mar 30, 2017

👍

@hweng hweng merged commit 4b09617 into develop Mar 30, 2017
@hweng hweng deleted the feature/nokogili_upgrade branch March 30, 2017 16:49
@mcritchlow mcritchlow mentioned this pull request Apr 3, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants