Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

unlock nokogiri so we can do minor updates #222

Merged
merged 1 commit into from
Mar 30, 2017
Merged

unlock nokogiri so we can do minor updates #222

merged 1 commit into from
Mar 30, 2017

Conversation

grosser
Copy link
Collaborator

@grosser grosser commented Jan 1, 2017

@arangamani

I want to use 1.7.x series ...

@grosser grosser mentioned this pull request Jan 1, 2017
Merged
3 tasks
@grosser
Copy link
Collaborator Author

grosser commented Jan 1, 2017

An error occurred while installing jeweler (2.3.2), and Bundler cannot continue.

... want a PR to remove jeweler ?

@grosser grosser mentioned this pull request Mar 24, 2017
Closed
@grosser
Copy link
Collaborator Author

grosser commented Mar 24, 2017

@arangamani if you don't maintain this anymore, add me as maintainer ?

@st33b
Copy link

st33b commented Mar 29, 2017

Seconding this PR. jenkins_api_client is the only thing holding us back from updating Nokogiri to 1.7.*. Thanks!

@grosser
Copy link
Collaborator Author

grosser commented Mar 29, 2017

@arangamani can you add another maintainer or merge/release this ?

@arangamani arangamani merged commit 3915b63 into arangamani:master Mar 30, 2017
@grosser
Copy link
Collaborator Author

grosser commented Mar 30, 2017

❤️

@arangamani
Copy link
Owner

Gem version 1.5.2 is released.

@grosser
Copy link
Collaborator Author

grosser commented Mar 30, 2017

that did not work out ... still depend on 1.6.0 :(

@grosser grosser deleted the grosser/no branch March 30, 2017 01:27
@grosser
Copy link
Collaborator Author

grosser commented Mar 30, 2017

... unable to run anything ... setup is broken / a mess ... are you ok with kicking out jeweler ?

@grosser
Copy link
Collaborator Author

grosser commented Mar 30, 2017

oh, you already did :D

@grosser
Copy link
Collaborator Author

grosser commented Mar 30, 2017

here #230

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@grosser @st33b @arangamani