Skip to content
This repository was archived by the owner on Aug 13, 2024. It is now read-only.

Update Gems#20

Open
Michal Wrobel (sparrovv) wants to merge 1 commit into
snowplow-archive:masterfrom
sparrovv:update-gems
Open

Update Gems#20
Michal Wrobel (sparrovv) wants to merge 1 commit into
snowplow-archive:masterfrom
sparrovv:update-gems

Conversation

@sparrovv

Copy link
Copy Markdown
Contributor
  • httparty
  • rspec

- httparty
- rspec
@snowplowcla

Copy link
Copy Markdown

Michal Wrobel (@sparrovv) has signed the Software Grant and Corporate Contributor License Agreement

@sparrovv

Copy link
Copy Markdown
Contributor Author

Anton Parkhomenko (@chuwy) Are there any other changes required?

@chuwy

Copy link
Copy Markdown
Contributor

Michal Wrobel (@sparrovv) nope, scheduling this! Thank you so much.

Comment thread iglu-ruby-client.gemspec
}

s.add_runtime_dependency "httparty", "<= 0.14.0"
s.add_runtime_dependency "httparty", "~> 0.15.0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This locks down the version too tightly.

Suggested change
s.add_runtime_dependency "httparty", "~> 0.15.0"
s.add_runtime_dependency "httparty", "~> 0.15"

@lukaso

Copy link
Copy Markdown

Any chance some of these changes can be merged. It's been a long time and we are supporting a forked version.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

4 participants