Skip to content

Conversation

@eak12913
Copy link
Contributor

This PR updated the dependency for the nokogiri library. This is a library used by the jekyll-toc plugin and is not used directly. The reason to update the version of nokogiri is because of a security notification from Dependabot. I don't believe that there's any issues from making this update.

This PR updated the dependency for the nokogiri library. This is a library used by the `jekyll-toc` plugin and is not used directly. The reason to update the version of nokogiri is because of a security notification from Dependabot. I don't believe that there's any issues from making this update.
@netlify
Copy link

netlify bot commented Jan 13, 2021

✔️ Deploy preview for keen-clarke-470db9 ready!

🔨 Explore the source changes: d3009e0

🔍 Inspect the deploy logs: https://app.netlify.com/sites/keen-clarke-470db9/deploys/5fff4ceab9cfe90007de97b5

😎 Browse the preview: https://deploy-preview-393--keen-clarke-470db9.netlify.app

@eak12913
Copy link
Contributor Author

@oredavids Could you please take a look here and approve if you think it looks good? Thanks!

Copy link
Contributor

@oredavids oredavids left a comment

Choose a reason for hiding this comment

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

LGTM & I don't see any errors in the console.

@oredavids oredavids merged commit a538f32 into master Jan 28, 2021
@oredavids oredavids deleted the update-nokogiri-dependency branch January 28, 2021 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants