-
Notifications
You must be signed in to change notification settings - Fork 252
chore: add scalafmt and re-format all the code #507
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@rjurney @bjornjorgensen Hello! Can you review it, please? It may look huge but there are no changes in the code, only formatting of the code. Thanks! |
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #507 +/- ##
==========================================
- Coverage 91.43% 90.73% -0.70%
==========================================
Files 18 18
Lines 829 896 +67
Branches 52 100 +48
==========================================
+ Hits 758 813 +55
- Misses 71 83 +12 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@SemyonSinchenko oh, one thing - are these the same settings as Spark? The plan is to eventually merge GraphFrames into Spark main, so consistency with their settings would be nice. |
I copied the whole |
@rjurney Can we merge this one? It allows to apply |
@rjurney are there anything wrong with this PR? |
Merging... @bjornjorgensen nothing wrong :) |
This PR closes #500
Changes:
.scalafmt.conf
is added with content exactly like in the Apache Spark Projectscalafmt
plugin is added to the plugins.metals
is added to the.gitignore
P.S. Metals is the most popular LSP for Scala that is used in VSCode, Vim / NeoVim, Emacs, Sublime, etc. I think it is nice to have it in
.gitignore
because it simplify the working with git and reduce a chance of accidentally pushing something not related to the project itself.P.P.S. All the changes in the scala code is related to formatting only! There are no actual changes of the logic.