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

Migrate examples project to Gradle subproject #1177

Merged
merged 11 commits into from
Feb 19, 2019
Merged

Conversation

kiview
Copy link
Member

@kiview kiview commented Jan 23, 2019

I'm not touching the docs here, this was just for migrating the existing Maven example project.
Let's see if it works on CircleCI also...

I thought it would be cute to have a sub-subproject structure, like :examples:spring-boot.
First I wanted to do it for our modules as well, but it would break the existing cross subproject dependencies, so I left it as is for modules.

Copy link
Member

@bsideup bsideup left a comment

Choose a reason for hiding this comment

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

Dependabot config needs to be updated as well

@bsideup
Copy link
Member

bsideup commented Jan 28, 2019

@kiview I would really recommend using includeBuild.
Otherwise, our examples are not copyable (testCompile project(":kafka") instead of testCompile 'org.testcontainers:kafka')

@kiview
Copy link
Member Author

kiview commented Jan 28, 2019

I wasn't aware of includeBuild, sounds good.

settings.gradle Outdated Show resolved Hide resolved
@bmuschko
Copy link
Contributor

@kiview Do you have a target date for getting this PR merged? Would love to contribute more examples.

@kiview
Copy link
Member Author

kiview commented Feb 13, 2019

I'd be good to merge, let's have @bsideup a final look 🙂

.travis.yml Show resolved Hide resolved
}

dependencies {
compileOnly 'org.slf4j:slf4j-api:1.7.25'
Copy link
Member

Choose a reason for hiding this comment

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

why is it compileOnly?

Copy link
Member Author

Choose a reason for hiding this comment

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

It was provided in the Maven pom.xml.
Let's just remove this.

redis.start();
}
public static final GenericContainer redis = new GenericContainer("redis:3.0.6")
.withExposedPorts(6379);
Copy link
Member

Choose a reason for hiding this comment

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

is it IntelliJ's formatting? o_O

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, or our .editorconfig?

@kiview
Copy link
Member Author

kiview commented Feb 19, 2019

@bsideup Are we good to merge?

@bsideup
Copy link
Member

bsideup commented Feb 19, 2019

@kiview yes 🚀

@kiview kiview added this to the next milestone Feb 19, 2019
@kiview kiview merged commit e0f361a into master Feb 19, 2019
@delete-merged-branch delete-merged-branch bot deleted the example-gradle-migration branch February 19, 2019 20:46
@kiview
Copy link
Member Author

kiview commented Feb 19, 2019

@bmuschko We just merged the PR, we'd love to get more examples if you have some good ideas 🙂

@bmuschko
Copy link
Contributor

@kiview Nice, I will check it out in the coming days.

@rnorth
Copy link
Member

rnorth commented Mar 12, 2019

@rnorth rnorth removed this from the next milestone Mar 12, 2019
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

4 participants