Bump Scalafmt to 3.9.x

[bowenliang123]

🔍 Description

Issue References 🔗

This pull request fixes #

Describe Your Solution 🔧

Without this PR:

• use and conform Scalafmt 3.7.x
• forcing all the imports from the same parent package, which is even violation the max length of the line

With this PR:

• use and conform Scalafmt 3.9.x
• Scalafmt 3.8.2 changes the binpack style: https://github.com/scalameta/scalafmt/releases/tag/v3.8.2
• change to binPack.importSelectors=singleLine, minimizing the impacts to existed code

Types of changes 🔖

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Behavior Without This Pull Request ⚰️

Behavior With This Pull Request 🎉

Related Unit Tests

---

Checklist 📝

• This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (ed96ac1) to head (d9f4f9d).
Report is 1 commits behind head on master.

Additional details and impacted files

@@          Coverage Diff           @@
##           master   #6674   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         689     689           
  Lines       42615   42615           
  Branches     5808    5808           
======================================
  Misses      42615   42615           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pan3793]

I'm surprised that scalafmt made such a breaking change. Maybe it's time to discuss Kyuubi 2.0, and we can consider including this change in 2.0 and keeping the existing behavior on 1.x serials.

[bowenliang123]

I agree. This PR is more about a demonstration for the impact of scalafmt 3.8, which is no hurry to be merged for now.

[bowenliang123]

Hi @pan3793 , I've reopened this PR and it should be ready for merge.
After further investigation latest changes in Scalafmt's configs, singleLine for binPack.importSelectors helps minimizing the impacts to our code, while the fold option would be a better option to conforming max columns. Config doc: binPack.importSelectors

[bowenliang123]

This PR is bringing less impact than previouly expected in earyly version of 3.8.x, and it's ready to go. WDYT? @pan3793

[pan3793]

too weird style ...

[bowenliang123]

Yes. Maybe we would find another way or config to get rid of it in the future. We are modernizing the config name into binPack.importSelectors this time.