Convert project to Maven and update dependencies #2

Merged
merged 8 commits into from Oct 10, 2013

Projects

None yet

2 participants

@leocomelli
Contributor

Converted project to Maven. Use it like a maven dependency on your project:

<dependency>
    <groupId>org.dbunit.dataset.builder</groupId>
    <artifactId>dbunit-datasetbuilder</artifactId>
    <version>1.0</version>
</dependency>

Updated dependencies
dbunit: 2.4.8 > 2.4.9
slf4j-*: 1.6.1 > 1.7.5
junit: 4.11

@marcphilipp marcphilipp and 1 other commented on an outdated diff Oct 8, 2013
@@ -0,0 +1,49 @@
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+ xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+
+ <modelVersion>4.0.0</modelVersion>
+ <groupId>org.dbunit.dataset.builder</groupId>
+ <artifactId>dbunit-datasetbuilder</artifactId>
+ <version>1.0</version>
@marcphilipp
marcphilipp Oct 8, 2013 Owner

This should be 1.0-SNAPSHOT, shouldn't it?

@leocomelli
leocomelli Oct 8, 2013 Contributor

Ok! I don't see any problem.

@marcphilipp marcphilipp and 1 other commented on an outdated diff Oct 8, 2013
@@ -0,0 +1,49 @@
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+ xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+
+ <modelVersion>4.0.0</modelVersion>
+ <groupId>org.dbunit.dataset.builder</groupId>
+ <artifactId>dbunit-datasetbuilder</artifactId>
+ <version>1.0</version>
+
+ <name>DBUnint Data Set Builder</name>
+ <description>DBUnit - Dynamically Creating Data Sets Using Builders</description>
+
+ <licenses>
+ <license>
+ <name>GNU General Public License (GPL)</name>
@marcphilipp
marcphilipp Oct 8, 2013 Owner

Originally, I had planned to integrate this code with the core DbUnit project. Thus, I decided for the same licence which at the time was LGPL. Has that changed?

@leocomelli
leocomelli Oct 8, 2013 Contributor

Sorry! error caused by copy and paste.

@marcphilipp
Owner

Looks very good, overall, thanks! Please give me a couple of days to review it thoroughly.

@marcphilipp marcphilipp merged commit 3743435 into marcphilipp:master Oct 10, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment