Skip to content

Commit

Permalink
Switch cacio back to parent project (#48)
Browse files Browse the repository at this point in the history
* Use cacio-tta 1.18 instead of patched fork

* Remove license exception for The Hyve fork of cacio-tta
  • Loading branch information
janblom authored Feb 27, 2024
1 parent 6fd6e5c commit ae5f416
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 14 deletions.
3 changes: 0 additions & 3 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -309,9 +309,6 @@
it is published on Maven repos -->
<exclude>com.microsoft.sqlserver:sqljdbc4</exclude>

<!-- temporary exclusion for the patched caciocavallo dependency -->
<exclude>com.github.thehyve:caciocavallo</exclude>

<!-- PLEASE NOTE: teradata and googlebigquery are provided within the project, they
cannot be found on a maven repo; no license info available; they should ideally
not be part of the project, and should be removed here -->
Expand Down
13 changes: 2 additions & 11 deletions whiterabbit/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -282,19 +282,10 @@
<version>5.2.5</version>
<scope>compile</scope>
</dependency>
<!-- cacio-tta cannot be used with JDK 18+. Until that is solved in the parent project, use a fixed
fork hosted at The Hyve -->
<!-- dependency>
<dependency>
<groupId>com.github.caciocavallosilano</groupId>
<artifactId>cacio-tta</artifactId>
<version>1.17.4-SNAPSHOT</version>
<scope>test</scope>
</dependency -->
<!-- fixed fork, enabled through the jitpack repository -->
<dependency>
<groupId>com.github.thehyve</groupId>
<artifactId>caciocavallo</artifactId>
<version>1.17.7</version>
<version>1.18</version>
<scope>test</scope>
</dependency>
<!-- https://mvnrepository.com/artifact/org.assertj/assertj-swing-junit -->
Expand Down

0 comments on commit ae5f416

Please sign in to comment.