Remove Json::ParseError assignment

[olivier-thatch]

#385 was merged and released in grape-entity 1.0.2 (cf. changelog).

However, the main grape gem also defines ParseError, which produces ugly warnings:

/Users/olivier/.rbenv/versions/4.0.2/lib/ruby/gems/4.0.0/gems/grape-entity-1.0.3/lib/grape_entity/json.rb:9: warning: already initialized constant JSON::ParseError
/Users/olivier/.rbenv/versions/4.0.2/lib/ruby/gems/4.0.0/gems/grape-3.2.0/lib/grape/json.rb:8: warning: previous definition of ParseError was here
/Users/olivier/.rbenv/versions/4.0.2/lib/ruby/gems/4.0.0/gems/grape-entity-1.0.3/lib/grape_entity/json.rb:9: warning: already initialized constant JSON::ParseError
/Users/olivier/.rbenv/versions/4.0.2/lib/ruby/gems/4.0.0/gems/grape-3.2.0/lib/grape/json.rb:8: warning: previous definition of ParseError was here

Since grape-entity makes no use of ParseError (cf. https://github.com/ruby-grape/grape-entity/pull/385/changes#r1594479678), we can drop the assignment entirely.

[github-actions]

Danger Report

No issues found.

View run

[numbata]

Thanks @olivier-thatch ! Good catch on the duplicate constant warnings.

This was also noted in PR that you mentioned. Since grape-entity doesn't do any JSON parsing, the ParseError assignment was unnecessary from the start.

Could you add a CHANGELOG entry as Danger suggests?