Skip to content

chore: Update .scalafmt.conf#364

Merged
octonato merged 4 commits intomainfrom
wip-scalafmt-patriknw
Sep 13, 2021
Merged

chore: Update .scalafmt.conf#364
octonato merged 4 commits intomainfrom
wip-scalafmt-patriknw

Conversation

@patriknw
Copy link
Copy Markdown
Contributor

Should be a decent time to do this now.

@patriknw patriknw force-pushed the wip-scalafmt-patriknw branch from 04f901b to 7ba5fd0 Compare September 13, 2021 08:45
Copy link
Copy Markdown
Contributor

@raboof raboof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now seems to be a good time, perhaps we want to go all the way to 3.0.3?

Comment thread .scalafmt.conf Outdated
@@ -1,24 +1,20 @@
version = 2.1.0
version = 2.7.5
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we go all the way to 3.0.3?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, I tried that, see f41a9ef and 0c6df07

@patriknw patriknw force-pushed the wip-scalafmt-patriknw branch from 7931f51 to 0c6df07 Compare September 13, 2021 09:51
/**
* INTERNAL API
* Used by the generated testkit
* INTERNAL API Used by the generated testkit
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, perhaps use an extra newline? Not important though.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, that wasn't so helpful formatting, but not important

@octonato octonato merged commit e2679d1 into main Sep 13, 2021
@octonato octonato deleted the wip-scalafmt-patriknw branch September 13, 2021 11:37
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