Skip to content
This repository was archived by the owner on May 5, 2026. It is now read-only.

Rails 5 support#7

Closed
chrisbutler wants to merge 2 commits into
mock-server:masterfrom
NationalResearchCorporation:rails-5
Closed

Rails 5 support#7
chrisbutler wants to merge 2 commits into
mock-server:masterfrom
NationalResearchCorporation:rails-5

Conversation

@chrisbutler
Copy link
Copy Markdown

hey @jamesdbloom, sorry for creating another PR (#6) but someone here accidentally deleted our fork of your repo

let me know if this PR needs any changes... it would be great to be able to switch back to using a version of your gem instead of a branch of someone else's fork!

Comment thread mockserver-client.gemspec Outdated
spec.add_dependency 'activesupport', '~> 4.1'
spec.add_dependency 'rest-client'
spec.add_dependency 'json', '>= 1.8'
spec.add_dependency 'activesupport', '>= 4.1'
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@chrisbutler I think the build here is failing b/c these dependencies were added twice:

spec.add_dependency 'activesupport', '>= 4.1'
spec.add_dependency 'rest-client', '>= 1.7'

- Locking the json to an old version seems uneccessary
- Forcing multi_json and json_pure monkeypatching broke our test suite
- removed the json_pure and so far so good
@chrisbutler
Copy link
Copy Markdown
Author

@jamesdbloom i made the fix @AJMiller pointed out, and everything is passing now. any chance we can get this merged in?

@jamesdbloom
Copy link
Copy Markdown
Collaborator

Closing as this PR is 8 years old and Rails 5 reached end-of-life in 2023 (current version is Rails 8). The Ruby client will be fully regenerated from the OpenAPI specification as part of monorepo consolidation, which will ensure compatibility with current Ruby and Rails versions. Thank you for your contribution @chrisbutler! See https://github.com/mock-server/mockserver for ongoing work.

@jamesdbloom
Copy link
Copy Markdown
Collaborator

Tracking issue created: mock-server/mockserver-monorepo#2051

Rails compatibility will be verified during Ruby client regeneration. Thank you for highlighting this need!

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.

4 participants