This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch branch-1.7
in repository https://gitbox.apache.org/repos/asf/orc.git


The following commit(s) were added to refs/heads/branch-1.7 by this push:
     new e773abb  ORC-882: Remove hamcrest-core test dependency (#788)
e773abb is described below

commit e773abb4881a04e94bd3f31a90e0bd5c87c380ee
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Fri Jul 30 08:17:39 2021 -0700

    ORC-882: Remove hamcrest-core test dependency (#788)
    
    ### What changes were proposed in this pull request?
    
    This PR aims to remove `hamcrest-core` test dependency.
    
    ### Why are the changes needed?
    
    This was added at ORC-60 and is not used now.
    ```
    $ git show 08b854425 | grep hamcrest
    +      <groupId>org.hamcrest</groupId>
    +      <artifactId>hamcrest-core</artifactId>
    +        <groupId>org.hamcrest</groupId>
    +        <artifactId>hamcrest-core</artifactId>
    ```
    
    ### How was this patch tested?
    
    Pass the CIs.
    
    This closes #778 .
    
    (cherry picked from commit 73a15c08a52955b356f65b7ca885541762251f49)
    Signed-off-by: Dongjoon Hyun <[email protected]>
---
 java/core/pom.xml | 5 -----
 java/pom.xml      | 6 ------
 2 files changed, 11 deletions(-)

diff --git a/java/core/pom.xml b/java/core/pom.xml
index 603895c..3213072 100644
--- a/java/core/pom.xml
+++ b/java/core/pom.xml
@@ -97,11 +97,6 @@
       <scope>test</scope>
     </dependency>
     <dependency>
-      <groupId>org.hamcrest</groupId>
-      <artifactId>hamcrest-core</artifactId>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
       <groupId>org.mockito</groupId>
       <artifactId>mockito-core</artifactId>
       <scope>test</scope>
diff --git a/java/pom.xml b/java/pom.xml
index e97dabc..fc9da9a 100644
--- a/java/pom.xml
+++ b/java/pom.xml
@@ -769,12 +769,6 @@
         <scope>test</scope>
       </dependency>
       <dependency>
-        <groupId>org.hamcrest</groupId>
-        <artifactId>hamcrest-core</artifactId>
-        <version>1.3</version>
-        <scope>test</scope>
-      </dependency>
-      <dependency>
         <groupId>org.mockito</groupId>
         <artifactId>mockito-core</artifactId>
         <version>3.7.0</version>

Reply via email to