Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@oyvindberg
Copy link
Owner

Summary

  • Extend bleep fmt to format both Scala and Java source files
  • Scala files: formatted with scalafmt (as before)
  • Java files: formatted with google-java-format
  • Add FetchGoogleJavaFormat to download the google-java-format all-deps jar from GitHub releases using coursier cache
  • Rename Scalafmt.scala to Fmt.scala to reflect its broader scope

Test plan

  • Run bleep fmt on a project with Java files
  • Verify Java files are formatted with google-java-format style
  • Verify Scala formatting still works as before

🤖 Generated with Claude Code

oyvindberg and others added 3 commits December 12, 2025 22:33
Generalize the fmt command to format both Scala and Java source files:
- Scala files: formatted with scalafmt (as before)
- Java files: formatted with google-java-format

Add FetchGoogleJavaFormat to download the google-java-format all-deps
jar from GitHub releases using coursier cache.

Rename Scalafmt to Fmt to reflect its broader scope.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Update google-java-format from 1.24.0 to 1.33.0 to fix NoSuchMethodError
  on JDK 24 (getDiagnostics API changed in internal javac APIs)
- Add required --add-exports JVM flags for JDK 16+ compatibility
- Run both Scala and Java formatters even if one fails, accumulating errors

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@oyvindberg oyvindberg merged commit 115ba93 into master Dec 13, 2025
7 checks passed
@oyvindberg oyvindberg deleted the feature/java-formatting branch December 13, 2025 21:53
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