-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Add new section for test integrations #6930
Conversation
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.
Thanks for the contribution! I've left some comments
Co-authored-by: Eddú Meléndez Gonzales <eddu.melendez@gmail.com>
Co-authored-by: Eddú Meléndez Gonzales <eddu.melendez@gmail.com>
Done :) |
Hi @LeoColman, thinking more about this... I think we should not duplicate the existing documentation in kotest but we should point to it. what do you think about renaming the file to integration and then add Kotest with a link to https://kotest.io/docs/extensions/test_containers.html The content can says something like |
I agree!
I even tried to make it smaller to avoid maintenance issues.
How could I apply your suggestion?
Apr 20, 2023 19:46:14 Eddú Meléndez Gonzales ***@***.***>:
…
Hi @LeoColman[https://github.com/LeoColman], thinking more about this... I think we should not duplicate the existing documentation in kotest but we should point to it. what do you think about renaming the file to integration and then add Kotest with a link to https://kotest.io/docs/extensions/test_containers.html
The content can says something like *The following open source projects use Testcontainers*
—
Reply to this email directly, view it on GitHub[#6930 (comment)], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AAMBCI7FIZCNKMDDT3MKGLLXCHDEDANCNFSM6AAAAAAXEV2VXY].
You are receiving this because you were mentioned.[Tracking image][https://github.com/notifications/beacon/AAMBCI6NH45AMXKBGEDNKN3XCHDEDA5CNFSM6AAAAAAXEV2VX2WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTS2NSL26.gif]
|
Just update this PR :) |
I thought of adding a |
Thanks for your contribution, @LeoColman ! This will be available soon! |
This commit adds documentation on how Testcontainers can be integrated with the Kotest framework.
From the very old #2533 it was decided that a reference to Kotest would be great in the docs. This commit adds it.