Skip to content

Apply spotless and checkstyle - #5390

Merged
eddumelendez merged 13 commits into
masterfrom
apply_spotless
May 25, 2022
Merged

eddumelendez merged 13 commits into
masterfrom
apply_spotless

Conversation

@eddumelendez

Copy link
Copy Markdown
Member
  • Apply spotless and checkstyle
  • Add preCommit

@eddumelendez
eddumelendez requested a review from a team May 20, 2022 23:26
@eddumelendez eddumelendez added this to the next milestone May 20, 2022
@eddumelendez eddumelendez changed the title apply spotless Apply spotless and checkstyle May 20, 2022
Comment thread build.gradle
dependencies {
// https://github.com/melix/japicmp-gradle-plugin/issues/36
classpath 'com.google.guava:guava:30.1.1-jre'
classpath 'com.github.kiview:captain-hook:76a1c11a16'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@kiview do you remember why we had to fork it?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

to make it work with windows I guess https://github.com/kiview/captain-hook/commits/master

@kiview kiview May 23, 2022 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Exactly, PR was never merged in upstream, since upstream seems to be dead 😞
tjni/captain-hook#4

Comment thread build.gradle
Comment thread core/src/main/java/org/testcontainers/DockerClientFactory.java Outdated
Comment thread core/src/main/java/org/testcontainers/DockerClientFactory.java Outdated
Comment thread modules/db2/src/main/java/org/testcontainers/containers/Db2Container.java Outdated
@eddumelendez

Copy link
Copy Markdown
Member Author

the in-docker_test fails because captain-hook try try to use git inside the docker image

Comment thread build.gradle Outdated
Comment thread core/src/main/java/org/testcontainers/containers/output/OutputFrame.java Outdated
Comment on lines +14 to +15
@Language("groovy")
String myTest = """

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Those lines should be aligned. But if it is hard to configure, we can live with that.

Comment thread core/src/test/java/org/testcontainers/junit/DockerfileContainerTest.java Outdated
Comment thread core/src/test/java/org/testcontainers/junit/DockerfileTest.java

@bsideup bsideup left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🎉 💯

@eddumelendez
eddumelendez merged commit de1324e into master May 25, 2022
@delete-merged-branch
delete-merged-branch Bot deleted the apply_spotless branch May 25, 2022 04:31
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.

3 participants