Skip to content

Pre-Scala 2.13: Remove deprecated ProcedureSyntax - use def x(): Unit = {...}#523

Merged
rtyley merged 1 commit intomainfrom
pre-scala-2.13-use-procedure-syntax
Aug 7, 2024
Merged

Pre-Scala 2.13: Remove deprecated ProcedureSyntax - use def x(): Unit = {...}#523
rtyley merged 1 commit intomainfrom
pre-scala-2.13-use-procedure-syntax

Conversation

@rtyley
Copy link
Copy Markdown
Member

@rtyley rtyley commented Aug 6, 2024

This change was automated with ScalaFix - in my ~/.sbt/1.0/plugins.sbt file, I added this line:

addSbtPlugin("ch.epfl.scala" % "sbt-scalafix" % "0.12.1")

Then at the sbt console ran:

scalafixEnable
scalafixAll ProcedureSyntax

...steps copied from https://github.com/guardian/content-api/pull/2893 & https://github.com/guardian/content-api/pull/2894 .

See also

In `~/.sbt/1.0/plugins.sbt`, added this line:

```
addSbtPlugin("ch.epfl.scala" % "sbt-scalafix" % "0.12.1")
```

Then at the sbt console ran:

```
scalafixEnable
scalafixAll ProcedureSyntax
```

...steps copied from guardian/content-api#2893 and
guardian/content-api#2894 .
@rtyley rtyley force-pushed the pre-scala-2.13-use-procedure-syntax branch from 4aade77 to 9dfce52 Compare August 6, 2024 17:00
@rtyley rtyley marked this pull request as ready for review August 6, 2024 17:03
@rtyley rtyley requested a review from JamieB-gu August 6, 2024 17:03
@rtyley rtyley merged commit e2462c4 into main Aug 7, 2024
@rtyley rtyley deleted the pre-scala-2.13-use-procedure-syntax branch August 7, 2024 09:32
@rtyley rtyley added Scala 2.13 See https://github.com/guardian/maintaining-scala-projects/issues/2 scalafix-rule-update Automatic rewrite using Scalafix (https://scalacenter.github.io/scalafix/) labels Aug 7, 2024
@rtyley rtyley mentioned this pull request Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scala 2.13 See https://github.com/guardian/maintaining-scala-projects/issues/2 scalafix-rule-update Automatic rewrite using Scalafix (https://scalacenter.github.io/scalafix/)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants