Skip to content

Replace yajl-ruby with the json gem for JSON handling#478

Merged
kenhys merged 1 commit into
fluent:masterfrom
Watson1978:yajl-to-json
Jun 10, 2026
Merged

Replace yajl-ruby with the json gem for JSON handling#478
kenhys merged 1 commit into
fluent:masterfrom
Watson1978:yajl-to-json

Conversation

@Watson1978

Copy link
Copy Markdown
Contributor

Since the yajl-ruby gem depends on a deprecated Ruby C API, it cannot be installed with Ruby 4.1.

Since the yajl-ruby gem depends on a deprecated Ruby C API,
it cannot be installed with Ruby 4.1.

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
@Watson1978

Copy link
Copy Markdown
Contributor Author

No remaining Yajl in codes.

$ grep -i "yajl" -r *
ChangeLog:  * Use Yajl instead of to_json not to raise exceptions when it got invalid bytes as UTF-8.

@Watson1978 Watson1978 requested a review from kenhys June 10, 2026 04:39

@kenhys kenhys left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@kenhys kenhys merged commit 61f24d3 into fluent:master Jun 10, 2026
16 of 17 checks passed
@Watson1978 Watson1978 deleted the yajl-to-json branch June 10, 2026 05:55
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.

2 participants