Skip to content

Update palantir-java-format to 2.100.0 and logback to 1.6.5 - #213

Merged
bernardladenthin merged 1 commit into
mainfrom
claude/hopeful-pascal-9jlbqb
Oct 1, 2026
Merged

bernardladenthin merged 1 commit into
mainfrom
claude/hopeful-pascal-9jlbqb

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

  • palantir-java-format 2.100.0 in all three module poms, and logback 1.6.5 in the parent's dependencyManagement (test scope only), matching the sibling repos.
  • CLAUDE.md dependency table updated.

Test plan

  • CI is green on this branch
  • Docs updated (CLAUDE.md)

Related issues / PRs

Part of the 2026-10-01 cross-repo dependency sweep (java-llama.cpp, BitcoinAddressFinder, streambuffer, BroomCabinet, workspace).

Checklist

  • No security-sensitive changes

🤖 Generated with Claude Code

https://claude.ai/code/session_01AytmJF9faEiQEVt6eetQS2


Generated by Claude Code

No formatting change under the new formatter; logback is test scope here. Verified with
mvn verify against a locally installed net.ladenthin:llama 5.2.0-SNAPSHOT (5.2.0 itself is still
unreleased, see TODO.md).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AytmJF9faEiQEVt6eetQS2
@claude

claude Bot commented Oct 1, 2026

Copy link
Copy Markdown

Review Summary ✅

This is a well-executed dependency update PR. All changes are consistent and properly documented.

Observations

✅ Version Updates

  • Logback 1.6.5 (patch security update) in parent dependencyManagement — correctly scoped to test scope only, respecting the Java 8 bytecode floor constraint
  • Palantir-java-format 2.100.0 applied consistently across all three module pom files (srcmorph, srcmorph-cli, srcmorph-maven-plugin)

✅ Documentation

  • CLAUDE.md dependency table updated to reflect logback 1.6.5 — no drift between code and docs

✅ Consistency

  • No module was missed; all changes are symmetric and aligned
  • Part of coordinated cross-repo update (matching sibling repos as noted in PR description)

✅ Security

  • Logback patch version is a security/maintenance release
  • Palantir-java-format (a code formatter) has minimal security surface
  • PR author correctly marked as no security-sensitive changes

Conclusion

Ready to merge once CI passes. Clean, consistent, low-risk dependency update.

@bernardladenthin
bernardladenthin merged commit 84c9c4d into main Oct 1, 2026
8 of 14 checks passed
@bernardladenthin
bernardladenthin deleted the claude/hopeful-pascal-9jlbqb branch October 1, 2026 16:58

This branch had an error being deployed

1 failed deployment
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