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

Fix some doc typos + slight doc proposal on java integration test section #3553

Merged
merged 5 commits into from
Sep 16, 2024

Conversation

NyuB
Copy link
Contributor

@NyuB NyuB commented Sep 15, 2024

Fixed some doc typos in the Java section,

  • Maven/Gradle confusion
  • Missing closing markdiwn quote
  • Dangling sentence

I also rephrased a section on java integration test, emphasizing more on the mill way to do it rather than the usage of Junit5/4, i can drop the commit if it is not suited

@@ -57,7 +57,7 @@ trait MicronautModule extends MavenModule{
// Although Mill does not have a built in `MicronautModule`, this example shows how easy
// it is to define it yourself as `trait MicronautModule`: setting up the annotation processor
// classpath as a `JavaModule` and setting up the annotation via `javacOptions`. Once defined,
// you can then use `MicronautModule in your build just like you.
// you can then use `MicronautModule` in your build just like you.
Copy link
Member

Choose a reason for hiding this comment

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

Is something missing at the end of this sentence? "just like you ..." What?

@lihaoyi lihaoyi merged commit 5dacd34 into com-lihaoyi:main Sep 16, 2024
24 checks passed
@lefou lefou added this to the 0.12.0 milestone Sep 16, 2024
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.

3 participants