scalameta/scalafmt

Code formatter for Scala

Tomasz Godzik

@tgodzik

I already have some ideas around it, but need to fix some mroe things in the parser

@2m:matrix.org

I am trying to use the latest scalafmt snapshot, but I am getting a following error:

Invalid version: 2.7.5+202-ace91064-SNAPSHOT

Should I format the version string a bit differently?

Tomasz Godzik

@tgodzik

I think that is due to the lack of resolver

not sure if it's possible to add the resolver to config

as a workaround you could use coursier to download the version from the sonatype snapshots repo

@2m:matrix.org

Eric K Richardson

@ekrich

@tgodzik I know you are super busy but have you had a chance to look at Scala Native scalafmt? Does it still use metaconfig? Maybe I could do a PR on that. Also, is paiges still used/getting updated anymore, looks a little old?

At one time Olafur was talking about a new formatter for scalafmt but said it was too much work - was that using paiges or pretty-print?

Tomasz Godzik

@tgodzik

@ekrich this is certainly on my mind, but I haven't looked into it yet :/ paiges seems not used anymore, I am not even sure what was it needed for

metaconfig on the other hand seems to be sued kind of all over the place

Eric K Richardson

@ekrich

I noticed Scala.js is commented out - do we need that for metaconfig or just native?

I can try and do that.

Tomasz Godzik

@tgodzik

no idea really, I only recently started working on it. I think JS was not that improtant

hmm... you meant that metaconfig was using paiges, maybe we can replace it with pprint

Eric K Richardson

@ekrich

Yeah, I see both in that build.

Let me see if I can get the native stuff in there and then we can go from there.

It was in there at one time.

Tomasz Godzik

@tgodzik

In the meantime I remove paiges from scalafmt scalameta/scalafmt#2398

Eric K Richardson

@ekrich

Ok, I commented out - we could update it - pprint now support 2.11 again thanks to jodersky probably which is really helpful.

The cli looks like it uses paiges - I could update if Olafur thinks it is worth it - did we give up on it and just use pprint?

Eric K Richardson

@ekrich

paiges is typelevel sorry - they dropped 2.11.

no native support

Tomasz Godzik

@tgodzik

We need to probably replace it with pprint for native - either repalce it altogether or just for native and create some compat classes

might be good to get some feedback from Olaf

Eric K Richardson

@ekrich

yeah, that is probably not that much - scalacheck-shapeless skips 2.11 for sjs 1 other stuff too.

Tomasz Godzik

@tgodzik

is shapeless used anywhere in scalafmt :O ?

Eric K Richardson

@ekrich

I think when lihaoyi dropped 2.11 it created a ripple downstream

Tomasz Godzik

@tgodzik

I don’t think paiges can easily be replaced with pprint.
It would be easier to inline the sources of paiges
Paiges has automated ci releases so it might also be worth trying to fix its cross-build.

citing Olaf

I asked him on discord

Eric K Richardson

@ekrich

Ok, it looks like typelevel has dropped 2.11 in release 0.4.0 would that be ok?

Sander Dijkhuis

@sander

How can I avoid the \n before _ in:

  class TsecLive[F[
      _]: Monad, HashAlgorithm, HmacAlgorithm, AuthCipher, AuthCipherSecretKey[
      _]](nonceSize: Int)(implicit
      val hasher: CryptoHasher[F, HashAlgorithm],

Tomasz Godzik

@tgodzik

@sander I think you can break if manually before F, which should put it all on the second line? We've been actually working on making long types work better, but that might come in 3.0.0

also not sure if it would solve your particular issue

Sander Dijkhuis

@sander

In this case that didn't work, possibly due to setting newlines { source = fold }

heksesang

@heksenlied:matrix.org

[m]

Is there a plan to release a version with Scala 3 support soon?

Tomasz Godzik

@tgodzik

@heksenlied:matrix.org Yes :D https://github.com/scalameta/scalafmt/releases/tag/v3.0.0-RC1

@/all We've just released Scalafmt with support for Scala 3 ! Not yet final release, but please do test it out!

Odd Möller

@odd

:clap:

Przemek Sokół

@falconepl

:clap:

heksesang

@heksenlied:matrix.org

[m]

Is there an option to remove braces when running with Scala3 dialect?

heksesang

@heksenlied:matrix.org

[m]

Also, I run into issues with errors about illegal start of simple expression at vals I declare in for yields like this: https://gitlab.com/heksesang/scala3-derivation/-/blob/master/pureconfig-derivation/src/no/heksesang/derivation/pureconfig/product.scala#L32

I assume it doesn't quite understand that I am simply omitting braces. It also doesn't like export statements.

Tomasz Godzik

@tgodzik

@heksenlied:matrix.org that will be available soon, probably in 3.0.0 final. For any issues please do report them.

heksesang

@heksenlied:matrix.org

[m]

How do I force newlines before multiline expressions in a for yield?

Tomasz Godzik

@tgodzik

@heksenlied:matrix.org unfold? https://scalameta.org/scalafmt/docs/configuration.html#newlinessourcefoldunfold

heksesang

@heksenlied:matrix.org

[m]

Doesn't seem to work.

heksesang

@heksenlied:matrix.org

[m]

before:

for {
  (id, user) <-
    create(
      givenName,
      surname,
      nin,
      memberships
    )
} yield user

after:

for {
  (id, user) <- create(
    givenName,
    surname,
    nin,
    memberships
  )
} yield user

1 reply

I want it to force the former.

But none of the options seem to do that.

Where communities thrive

People

Repo info

Activity