Skip to content
This repository has been archived by the owner on Jun 26, 2023. It is now read-only.

Upstream changes 20160418 #1

Merged
merged 5 commits into from
Apr 19, 2016
Merged

Upstream changes 20160418 #1

merged 5 commits into from
Apr 19, 2016

Conversation

johnsyweb
Copy link

Context

I'd like to use a later version of Nokogiri to test a fix for sparklemotion/nokogiri#1426

Change

  • Pull in upstream changes
  • Relax nokogiri version

Considerations

Should we send these changes upstream?

Cc

@mariovisic & @jacobbednarz

@jacobbednarz
Copy link

Makes sense! 🚢

@@ -1,3 +1,3 @@
module Wraith
VERSION = "3.1.2"
VERSION = "3.1.4"

Choose a reason for hiding this comment

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

Mmm, pi!

Copy link
Author

Choose a reason for hiding this comment

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

π!

@asellitt
Copy link

+1

1 similar comment
@mariovisic
Copy link

👍

@mariovisic
Copy link

Can I also suggest sending a PR to the project with the relaxed nokogiri version and an explanation as to why it's useful?

@johnsyweb
Copy link
Author

@mariovisic : You can indeed! bbc#425

@johnsyweb johnsyweb merged commit a4566a8 into master Apr 19, 2016
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.

5 participants