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.

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

Update jsonapi runtime dependency to 0.1.1.beta6 #2005

Merged
merged 1 commit into from
Dec 23, 2016

Conversation

kofronpi
Copy link
Contributor

Purpose

Support ruby 2.4

Changes

Update jsonapi runtime dependency to 0.1.1.beta6

Related GitHub issues

#2004

Thanks for the review 👍

@mention-bot
Copy link

@kofronpi, thanks for your PR! By analyzing the history of the files in this pull request, we identified @beauby, @bf4 and @dubadub to be potential reviewers.

@beauby
Copy link
Contributor

beauby commented Dec 16, 2016

@bf4 What's the deal with rails 5 and that test?

Once that's solved, we can merge this PR.

@bf4
Copy link
Member

bf4 commented Dec 18, 2016

@beauby this appears to have been broken by rails/rails#23810

EDIT 2016-12-23: I was wrong about this. This was a change ONLY in the active record JSON coder. The actual problem was that JSON.parse('') changed exception messages across some versions of the JSON gem.

@beauby
Copy link
Contributor

beauby commented Dec 18, 2016

@bf4 Let's merge this then?

@bf4
Copy link
Member

bf4 commented Dec 18, 2016

@beauby well, we still needs tests passing, and I was unable to duplicate the failure locally...
\

@beauby
Copy link
Contributor

beauby commented Dec 18, 2016

@bf4 I'm not convinced this PR is breaking the tests though.

bf4 added a commit to bf4/active_model_serializers that referenced this pull request Dec 19, 2016
@bf4
Copy link
Member

bf4 commented Dec 19, 2016

@beauby then we need to make a test PR to validate that hypothesis, no? #2007

@bf4
Copy link
Member

bf4 commented Dec 23, 2016

@beauby you're right that it's just a ruby version, but weird that #2007 passed...

screen shot 2016-12-23 at 11 08 06 am

So, just a bad expectation

@bf4
Copy link
Member

bf4 commented Dec 23, 2016

Merged in fix #2007 and then just merged this! 🎉 @beauby @kofronpi Thanks for sticking with this!

@bf4
Copy link
Member

bf4 commented Dec 23, 2016

Anyone wanna fix all those minitest warnings on Ruby 2.3.3 and Rails 5?

@williamweckl
Copy link

Any expectations to release it as a version?

bf4 added a commit that referenced this pull request Jan 6, 2017
Update jsonapi runtime dependency to 0.1.1.beta6
@bf4
Copy link
Member

bf4 commented Jan 6, 2017

@williamweckl @kofronpi @beauby made a PR to prepare a '0-10-stable' release #2016

bf4 added a commit that referenced this pull request Jan 6, 2017
Update jsonapi runtime dependency to 0.1.1.beta6
bf4 added a commit that referenced this pull request Jan 6, 2017
Update jsonapi runtime dependency to 0.1.1.beta6
bf4 added a commit that referenced this pull request Jan 6, 2017
Update jsonapi runtime dependency to 0.1.1.beta6
@bf4
Copy link
Member

bf4 commented Jan 6, 2017

0.10.4 is now released https://twitter.com/rubygems/status/817496467611013121 #2018

@williamweckl
Copy link

Thank you!

bf4 added a commit that referenced this pull request Jan 10, 2017
* Merge pull request #1990 from mxie/mx-result-typo

Fix typos and capitalization in Relationship Links docs [ci skip]

* Merge pull request #1992 from ojiry/bump_ruby_versions

Run tests by Ruby 2.2.6 and 2.3.3

* Merge pull request #1994 from bf4/promote_architecture

Promote important architecture description that answers a lot of questions we get
Conflicts:
	docs/ARCHITECTURE.md

* Merge pull request #1999 from bf4/typos

Fix typos [ci skip]

* Merge pull request #2000 from berfarah/patch-1

Link to 0.10.3 tag instead of `master` branch

* Merge pull request #2007 from bf4/check_ci

Test was failing due to change in JSON exception message when parsing empty string

* Swap out KeyTransform for CaseTransform (#1993)

* delete KeyTransform, use CaseTransform

* added changelog

Conflicts:
	CHANGELOG.md

* Merge pull request #2005 from kofronpi/support-ruby-2.4

Update jsonapi runtime dependency to 0.1.1.beta6

* Bump to v0.10.4

* Merge pull request #2018 from rails-api/bump_version

Bump to v0.10.4 [ci skip]
Conflicts:
	CHANGELOG.md

* Merge pull request #2019 from bf4/fix_method_redefined_warning

Fix AMS warnings

* Merge pull request #2020 from bf4/silence_grape_warnings

Silence Grape warnings

* Merge pull request #2017 from bf4/remove_warnings

Fix mt6 assert_nil warnings

* Updated isolated tests to assert correct behavior. (#2010)

* Updated isolated tests to assert correct behavior.
* Added check to get unsafe params if rails version is great than 5

* Merge pull request #2012 from bf4/cleanup_isolated_jsonapi_renderer_tests_a_bit

Cleanup assertions in isolated jsonapi renderer tests a bit

* Add Model#attributes helper; make test attributes explicit

* Fix model attributes accessors

* Fix typos

* Randomize testing of compatibility layer against regressions

* Test bugfix

* Add CHANGELOG

* Merge pull request #1981 from groyoh/link_doc

Fix relationship links doc
Conflicts:
	CHANGELOG.md
GregPK pushed a commit to GregPK/active_model_serializers that referenced this pull request Apr 25, 2017
* Merge pull request rails-api#1990 from mxie/mx-result-typo

Fix typos and capitalization in Relationship Links docs [ci skip]

* Merge pull request rails-api#1992 from ojiry/bump_ruby_versions

Run tests by Ruby 2.2.6 and 2.3.3

* Merge pull request rails-api#1994 from bf4/promote_architecture

Promote important architecture description that answers a lot of questions we get
Conflicts:
	docs/ARCHITECTURE.md

* Merge pull request rails-api#1999 from bf4/typos

Fix typos [ci skip]

* Merge pull request rails-api#2000 from berfarah/patch-1

Link to 0.10.3 tag instead of `master` branch

* Merge pull request rails-api#2007 from bf4/check_ci

Test was failing due to change in JSON exception message when parsing empty string

* Swap out KeyTransform for CaseTransform (rails-api#1993)

* delete KeyTransform, use CaseTransform

* added changelog

Conflicts:
	CHANGELOG.md

* Merge pull request rails-api#2005 from kofronpi/support-ruby-2.4

Update jsonapi runtime dependency to 0.1.1.beta6

* Bump to v0.10.4

* Merge pull request rails-api#2018 from rails-api/bump_version

Bump to v0.10.4 [ci skip]
Conflicts:
	CHANGELOG.md

* Merge pull request rails-api#2019 from bf4/fix_method_redefined_warning

Fix AMS warnings

* Merge pull request rails-api#2020 from bf4/silence_grape_warnings

Silence Grape warnings

* Merge pull request rails-api#2017 from bf4/remove_warnings

Fix mt6 assert_nil warnings

* Updated isolated tests to assert correct behavior. (rails-api#2010)

* Updated isolated tests to assert correct behavior.
* Added check to get unsafe params if rails version is great than 5

* Merge pull request rails-api#2012 from bf4/cleanup_isolated_jsonapi_renderer_tests_a_bit

Cleanup assertions in isolated jsonapi renderer tests a bit

* Add Model#attributes helper; make test attributes explicit

* Fix model attributes accessors

* Fix typos

* Randomize testing of compatibility layer against regressions

* Test bugfix

* Add CHANGELOG

* Merge pull request rails-api#1981 from groyoh/link_doc

Fix relationship links doc
Conflicts:
	CHANGELOG.md
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.

5 participants