Skip to content

Add scalafix rules for 0.13.0 release#330

Merged
zainab-ali merged 2 commits into
typelevel:mainfrom
zainab-ali:scalafix-0.13.0
Jun 1, 2026
Merged

Add scalafix rules for 0.13.0 release#330
zainab-ali merged 2 commits into
typelevel:mainfrom
zainab-ali:scalafix-0.13.0

Conversation

@zainab-ali
Copy link
Copy Markdown
Collaborator

@zainab-ali zainab-ali commented Jun 1, 2026

The 0.13.0 rule addresses the renames in #318 and #317.

This can be added to scala-steward via this config change.

Note that the 0.11.x and 0.9.x rule tests fail due to incorrectly applied formatting. scalafmt upgrades don't seem to respect our excludePaths. This can be addressed in #331 .

@zainab-ali zainab-ali marked this pull request as ready for review June 1, 2026 14:54
def basicFunction(suite: FunSuite): FunSuite = suite
}

import weaver.{ FunSuite => FunSuiteIOAlias }
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.

Nit: how about removing IO from the alias?

Suggested change
import weaver.{ FunSuite => FunSuiteIOAlias }
import weaver.{ FunSuite => FunSuiteAlias }

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I think it would be better to leave it untouched. We can't guarantee what the alias name is, nor whether it will conflict with anything else in the user code.

@zainab-ali zainab-ali merged commit 7e311e3 into typelevel:main Jun 1, 2026
13 checks passed
@zainab-ali zainab-ali deleted the scalafix-0.13.0 branch June 1, 2026 16:35
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.

2 participants