Skip to content

Rename mutable suites to type alias names.#318

Merged
zainab-ali merged 2 commits into
typelevel:mainfrom
zainab-ali:rename-mutable-suite
Jun 1, 2026
Merged

Rename mutable suites to type alias names.#318
zainab-ali merged 2 commits into
typelevel:mainfrom
zainab-ali:rename-mutable-suite

Conversation

@zainab-ali
Copy link
Copy Markdown
Collaborator

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

The weaver IOSuite and SimpleIOSuite are type aliases to MutableIOSuite and SimpleMutableIOSuite respectively.

These aliases were introduced as part of cats effect 3 support, when the codebase supported multiple effect types.

The codebase now only supports IO on cats effect 3, so these aliases may no longer be useful. This changeset renames the underlying Mutable* classes to these aliases.

TODO

  • Add a scalafix rule renamingMutableIOSuite to IOSuite
  • Remove MutableIOSuite from the docs.
  • Remove the Mutable name from any internal tests.

@zainab-ali zainab-ali force-pushed the rename-mutable-suite branch from dec9d75 to 17079aa Compare May 12, 2026 11:43
@zainab-ali zainab-ali force-pushed the rename-mutable-suite branch from 5f2c38a to f1bb79b Compare May 29, 2026 12:08
@zainab-ali zainab-ali marked this pull request as ready for review May 29, 2026 14:04
@zainab-ali
Copy link
Copy Markdown
Collaborator Author

I'll add the scalafix rule in a follow-up PR.

@zainab-ali zainab-ali merged commit ff3228c into typelevel:main Jun 1, 2026
13 checks passed
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