Skip to content
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

Convert Docs for the Junit5 #2977

Merged
merged 14 commits into from
Jul 12, 2020
Merged

Convert Docs for the Junit5 #2977

merged 14 commits into from
Jul 12, 2020

Conversation

raynigon
Copy link
Contributor

@raynigon raynigon commented Jul 6, 2020

Convert inline example code to use codeinclude macro for the JUnit5 Testframework documentation.
See #1158 for more details.

Convert inline example code to use codeinclude macro for the kafka module documentation.
Implement the codeinclude plugin for the junit5 docs
@raynigon raynigon changed the title WIP: Convert Docs for the Junit5 Convert Docs for the Junit5 Jul 7, 2020
@@ -0,0 +1,37 @@
package org.testcontainers.junit.jupiter;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

unfortunately i had to create a new test class, to have the code representing nearly the same logic as the previous documentation

@raynigon
Copy link
Contributor Author

raynigon commented Jul 9, 2020

Difference in the docs, left (old) contains the inline code, right (new) was generated from the unit tests.
junit

Copy link
Member

@rnorth rnorth left a comment

Choose a reason for hiding this comment

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

One tiny tweak - I'll apply it, but this is awesome @raynigon, thank you very much.

@rnorth rnorth merged commit 74a9240 into testcontainers:master Jul 12, 2020
@rnorth
Copy link
Member

rnorth commented Jul 12, 2020

Merged - thanks @raynigon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants