Skip to content

GitHub client runs on Jackson 2.21 - #39

Merged
akwirick merged 1 commit into
mainfrom
aw/jackson-snake-case-only
Nov 4, 2026
Merged

akwirick merged 1 commit into
mainfrom
aw/jackson-snake-case-only

Conversation

@akwirick

@akwirick akwirick commented Nov 4, 2026

Copy link
Copy Markdown

Changes:

Related: https://github.com/cortexapps/brain-backend/pull/19193 (blocked on this). Replaces the Jackson half of #38; the upstream merge there is a separate question.

Session: kotlin-vs-ts-dbos-rewrite (3c067139) · Worktree: github-api/jackson-only

🤖 Generated with Claude Code

Changes:
- Replaced the removed PropertyNamingStrategy.SNAKE_CASE with PropertyNamingStrategies.SNAKE_CASE in GitHubClient
- Bumped jackson-databind to 2.21.8
- Set the version to 1.331, since unmerged #37 already published 1.330
- Gated the deploy job to pushes on main, so pull request runs stop publishing

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@akwirick
akwirick requested a review from quicksnap November 4, 2026 00:33
@akwirick
akwirick merged commit 18cf347 into main Nov 4, 2026
9 checks passed
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