-
Notifications
You must be signed in to change notification settings - Fork 133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename the zio-kafka-test-utils
module to zio-kafka-testkit
&& change its base package from zio.kafka
to zio.kafka.testkit
#845
Conversation
713c1bd
to
65c864a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed with Erik on the library name change being inconvenient for users. Test kit / utils is quite similar. The namespace change is good though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@guizmaii convinced me that we can make this change; only power-users have found their way to test-utils yet. Together with proper documentation, we can make the switch.
@svroonland See my answer to Erik on Discord: https://discord.com/channels/629491597070827530/629497941719121960/1107265058695815168 Let me know what you think :) |
3c1a2e8
to
2e72287
Compare
zio-kafka-test-utils
module to zio-kafka-testkit
to make it more explicit that it's something that can be used by zio-kafka userszio-kafka-test-utils
module to zio-kafka-testkit
&& change its base package from zio.kafka
to zio.kafka.testkit
2e72287
to
6bc6ace
Compare
…ke it more explicit that it's something that can be used by zio-kafka users Also, change the base namespace of this module from `zio.kafka` to `zio.kafka.testkit` to make things more explicit
6bc6ace
to
e1613c4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one comment. I am fine with merging this PR with #846.
With the new name, I want to make this package more easily discoverable by our users.
The package change is to better separate things between zio-kafka and its testkit