Update formatters to latest version. Move versions to catalog. Minor reformatting of code. - #12166
Conversation
…reformatting of code.
| "scenario | objectKey | eTag | expectedHash ", | ||
| "basic values | some-key.data | ab12ef34 | e721375466d4116ab551213fdea08413", | ||
| "non-ascii key | some-key.你好 | ab12ef34 | d1333a04b9928ab462b5c6cadfa401f4 ", | ||
| "non-ascii key | some-key.你好 | ab12ef34 | d1333a04b9928ab462b5c6cadfa401f4", |
There was a problem hiding this comment.
TIL we have chinese in the repo
@sarahchen6 Yep, that was my fault, I forgot to provide that AKKA key locally |
|
🎯 Code Coverage (details) 🔗 Commit SHA: 96bbdb4 | Docs | Datadog PR Page | Give us feedback! |
There was a problem hiding this comment.
More details
The formatter catalog resolves from Kotlin DSL, Groovy DSL, applied scripts, child projects, and a reused configuration cache; whitespace-only source comparisons and version-alignment checks found no behavioral hazard.
📊 Validated against 10 scenarios · Open Bits AI session
🤖 Datadog Autotest · Commit da9008b · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
What Does This Do
Update formatters to latest version. Move versions to catalog. Minor reformatting of code.
Motivation
Keep latest tooling up to date.
Simplify future updates.
Additional Notes
Latest Scala formatter produced a little bit of noise, but not that much.
Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]