Skip to content

Rename io packages to iolocal#752

Merged
catostrophe merged 3 commits intotrace4cats:masterfrom
ybasket:rename-io-package
May 8, 2022
Merged

Rename io packages to iolocal#752
catostrophe merged 3 commits intotrace4cats:masterfrom
ybasket:rename-io-package

Conversation

@ybasket
Copy link
Copy Markdown
Contributor

@ybasket ybasket commented May 7, 2022

Rename packages as "io" is a common first segment of package names (io.circe for example) and can hence be very annoying to users that then have to sort their imports or prefix them with _root_.

Open question: Shall the module also be renamed from io to iolocal to keep naming consistent?

Rename packages as "io" is a common first segment of package names (io.circe for example) and can hence be very annoying to users that then have to sort their imports or prefix them with _root_.
@ybasket ybasket requested a review from a team May 7, 2022 11:44
@catostrophe
Copy link
Copy Markdown
Member

Yes, let's rename the package. I thought about it.

Yannick Heiber added 2 commits May 8, 2022 18:13
Ensure consistency with the previously renamed package names that avoided import clashs in user code.
@catostrophe catostrophe merged commit bcd8fd9 into trace4cats:master May 8, 2022
@catostrophe catostrophe added the enhancement New feature or request label May 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants