Skip to content

Conversation

davidangb
Copy link
Contributor

@davidangb davidangb commented Oct 25, 2024

This PR has two commits:

  1. Enable the scalafmt plugin and define a scalafmt conf; the conf is copied from Rawls https://github.com/broadinstitute/rawls/blob/develop/.scalafmt.conf
  2. Run scalafmt on all files. This second commit is large and is machine-generated, no changes by hand. You could choose to not review this commit.

Once this PR is merged, I will follow up with another PR (in draft at #1467) to add enforcement and to ignore the big reformatting in git blame.

@davidangb davidangb marked this pull request as ready for review October 25, 2024 17:54
@davidangb davidangb changed the title Scalafmt CORE-126: Enable scalafmt and format all source code Oct 25, 2024
@davidangb davidangb requested review from a team, calypsomatic and kevinmarete and removed request for a team October 25, 2024 17:57
@davidangb davidangb merged commit 3686bb4 into develop Oct 25, 2024
13 checks passed
@davidangb davidangb deleted the da_scalaFmtAll branch October 25, 2024 18:06
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.

3 participants