Skip to content

chore: pin json gem to < 3.0 to fix active_support conflict - #252

Merged
avsej merged 1 commit into
couchbase:mainfrom
avsej:fix-json-quirks-mode
Sep 9, 2026
Merged

avsej merged 1 commit into
couchbase:mainfrom
avsej:fix-json-quirks-mode

Conversation

@avsej

@avsej avsej commented Sep 9, 2026

Copy link
Copy Markdown
Member

Resolves ArgumentError: unknown keyword: quirks_mode in tests caused by json 3.0.2 removing support for quirks_mode, which activesupport-7.0.10 still relies on.

Pins json to < 3.0 in development to prevent this.

@avsej
avsej requested a balanced review from Copilot September 9, 2026 16:34
@avsej avsej added the rubber stamp needed Reviewed & ready internal/infrastructure change. Needs light approval. No public API impact. label Sep 9, 2026
@avsej
avsej requested a review from DemetrisChr September 9, 2026 16:34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused dependency constraint directly prevents the documented ActiveSupport incompatibility.

Pull request overview

Pins the development JSON dependency to maintain compatibility with ActiveSupport tests.

Changes:

  • Restricts json to versions below 3.0.
File summaries
File Description
Gemfile Adds the development-only JSON version constraint.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@avsej
avsej force-pushed the fix-json-quirks-mode branch from b899520 to 06719a6 Compare September 9, 2026 16:37
@avsej
avsej merged commit 2877f34 into couchbase:main Sep 9, 2026
34 of 44 checks passed
@avsej
avsej deleted the fix-json-quirks-mode branch September 9, 2026 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rubber stamp needed Reviewed & ready internal/infrastructure change. Needs light approval. No public API impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants