Skip to content

Conversation

@chris-zen
Copy link

No description provided.

* @param topic the topic name
* @param topicConfig per topic configuration [[Map]]
* @param config an implicit [[EmbeddedKafkaConfig]]
*/
Copy link
Owner

Choose a reason for hiding this comment

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

Can you please add some ScalaDocs here?

@manub
Copy link
Owner

manub commented May 27, 2016

It would be good also to update the README file with those changes.

@chris-zen
Copy link
Author

@manub I updated the ScalaDocs and the README, please let me know if there is anything else that need to be addressed ;-)

@manub manub merged commit 9ffb040 into manub:master May 30, 2016
@manub
Copy link
Owner

manub commented May 30, 2016

Planning to release this during the weekend.

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