Skip to content

Add scalafmt to recommendations - #205

Merged
jonathonherbert merged 1 commit into
mainfrom
jsh/recommend-scalafmt
Jul 15, 2026
Merged

Add scalafmt to recommendations#205
jonathonherbert merged 1 commit into
mainfrom
jsh/recommend-scalafmt

Conversation

@jonathonherbert

Copy link
Copy Markdown
Contributor

What is being recommended?

Add scalafmt to recommendations, with a little more structure to scope toolchain choices vs. writing Scala.

What's the context?

We've no linting/formatting recommendations for Scala, but we do have them for Javascript. Linting and formatting recommendations are great for repo hygiene.

@johnduffell johnduffell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

seems like a glaring omission nowadays! thanks for adding.

worth adding a note to run scalafmtCheck in CI?

@jonathonherbert

Copy link
Copy Markdown
Contributor Author

@johnduffell I'll leave it here for now, as scalafmt's documentation can give guidance on its usage — but if you do feel that'd be useful, alongside perhaps guidance on pre-push hooks etc., perhaps we could follow up in another PR that covers its usage more thoroughly?

@jonathonherbert
jonathonherbert merged commit 4678bd9 into main Jul 15, 2026
4 checks passed
@jonathonherbert
jonathonherbert deleted the jsh/recommend-scalafmt branch July 15, 2026 12:51
@shtukas

shtukas commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Late to the party but +1 to this!

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.

5 participants