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

albumenj pushed a commit to branch 3.1
in repository https://gitbox.apache.org/repos/asf/dubbo.git


The following commit(s) were added to refs/heads/3.1 by this push:
     new bb53fde978 Add dependencies check (#10896)
bb53fde978 is described below

commit bb53fde97845aff0a2a2dfd7cbb77fba21dfc316
Author: Albumen Kevin <[email protected]>
AuthorDate: Tue Nov 8 18:48:48 2022 +0800

    Add dependencies check (#10896)
---
 .github/workflows/build-and-test-pr.yml            |  16 +++
 .licenserc.yaml                                    | 115 +++++++++++++++++++++
 .../org/apache/dubbo/rpc/support/RpcUtilsTest.java |   4 +-
 3 files changed, 133 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/build-and-test-pr.yml 
b/.github/workflows/build-and-test-pr.yml
index 2d76db5fc8..28c669e39d 100644
--- a/.github/workflows/build-and-test-pr.yml
+++ b/.github/workflows/build-and-test-pr.yml
@@ -26,6 +26,22 @@ jobs:
         uses: apache/skywalking-eyes@main
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+      - name: "Set up JDK 17"
+        uses: actions/setup-java@v3
+        with:
+          distribution: 'zulu'
+          java-version: 17
+      - name: "Compile Dubbo (Linux)"
+        run: |
+          ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress 
--fail-fast -T 2C clean install -DskipTests=true -DskipIntegrationTests=true 
-Dcheckstyle.skip=true -Dcheckstyle_unix.skip=true -Drat.skip=true 
-Dmaven.javadoc.skip=true
+      - name: Check Dependencies' License
+        uses: apache/skywalking-eyes/dependency@main
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        with:
+          config: .licenserc.yaml
+          mode: check
+
   build-source:
     runs-on: ubuntu-latest
     outputs:
diff --git a/.licenserc.yaml b/.licenserc.yaml
index 00b7286672..fcbf54b12b 100644
--- a/.licenserc.yaml
+++ b/.licenserc.yaml
@@ -80,3 +80,118 @@ header:
   comment: on-failure
 
   license-location-threshold: 130
+
+dependency:
+  files:
+    - pom.xml
+    - dubbo-dependencies-bom/pom.xml
+  licenses:
+    - name: com.alibaba.spring:spring-context-support
+      license: Apache-2.0
+    - name: com.fasterxml.jackson.core:jackson-annotations
+      license: Apache-2.0
+    - name: com.fasterxml.jackson.core:jackson-core
+      license: Apache-2.0
+    - name: com.fasterxml.jackson.core:jackson-databind
+      license: Apache-2.0
+    - name: com.fasterxml.jackson.dataformat:jackson-dataformat-yaml
+      license: Apache-2.0
+    - name: com.fasterxml.jackson.datatype:jackson-datatype-jsr310
+      license: Apache-2.0
+    - name: com.google.code.gson:gson
+      license: Apache-2.0
+    - name: com.google.guava:listenablefuture
+      license: Apache-2.0
+    - name: com.salesforce.servicelibs:grpc-contrib
+      license: BSD 3-clause
+    - name: com.squareup.okhttp3:logging-interceptor
+      license: Apache-2.0
+    - name: com.squareup.okhttp3:okhttp
+      license: Apache-2.0
+    - name: com.squareup.okio:okio
+      license: Apache-2.0
+    - name: com.sun.xml.fastinfoset:FastInfoset
+      license: Apache-2.0
+    - name: io.envoyproxy.controlplane:api
+      license: Apache-2.0
+    - name: io.swagger:swagger-annotations
+      license: Apache-2.0
+    - name: io.swagger:swagger-models
+      license: Apache-2.0
+    - name: org.springframework.boot:spring-boot
+      license: Apache-2.0
+    - name: org.springframework.boot:spring-boot-actuator
+      license: Apache-2.0
+    - name: org.springframework.boot:spring-boot-autoconfigure
+      license: Apache-2.0
+    - name: org.springframework.boot:spring-boot-configuration-processor
+      license: Apache-2.0
+    - name: org.springframework.boot:spring-boot-starter
+      license: Apache-2.0
+    - name: org.springframework.boot:spring-boot-starter-actuator
+      license: Apache-2.0
+    - name: org.springframework.boot:spring-boot-starter-logging
+      license: Apache-2.0
+    - name: org.springframework.boot:spring-boot-starter-tomcat
+      license: Apache-2.0
+    - name: org.springframework.boot:spring-boot-starter-web
+      license: Apache-2.0
+    - name: org.slf4j:slf4j-api
+      license: MIT
+    - name: org.slf4j:slf4j-log4j12
+      license: MIT
+    - name: org.jboss.resteasy:resteasy-jaxrs
+      license: Apache-2.0
+    - name: org.jboss.resteasy:resteasy-client
+      license: Apache-2.0
+    - name: org.jboss.resteasy:resteasy-netty4
+      license: Apache-2.0
+    - name: org.jboss.resteasy:resteasy-jdk-http
+      license: Apache-2.0
+    - name: org.jboss.resteasy:resteasy-jackson-provider
+      license: Apache-2.0
+    - name: org.jboss.resteasy:resteasy-jaxb-provider
+      license: Apache-2.0
+    - name: net.jcip:jcip-annotations
+      license: Apache-2.0
+    - name: org.apache.zookeeper:zookeeper
+      license: Apache-2.0
+    - name: org.apache.zookeeper:zookeeper-jute
+      license: Apache-2.0
+    - name: net.bytebuddy:byte-buddy
+      license: Apache-2.0
+    - name: javax.enterprise:cdi-api
+      license: Apache-2.0
+    - name: org.codehaus.plexus:plexus-component-annotations
+      license: Apache-2.0
+    - name: org.slf4j:jcl-over-slf4j
+      license: Apache-2.0
+    - name: org.slf4j:jul-to-slf4j
+      license: Apache-2.0
+    # multi license
+    - name: org.javassist:javassist
+      license: Apache-2.0
+    - name: javax.annotation:javax.annotation-api
+      license: CDDL-1.0
+    - name: com.salesforce.servicelibs:jprotoc
+      license: CDDL-1.0
+    - name: org.checkerframework:checker-compat-qual
+      license: MIT
+    - name: ch.qos.logback:logback-classic
+      license: EPL-1.0
+    - name: ch.qos.logback:logback-core
+      license: EPL-1.0
+    - name: javax.servlet:javax.servlet-api
+      license: CDDL-1.1
+    - name: com.sun.activation:javax.activation
+      license: CDDL-1.1
+    - name: javax.activation:activation
+      license: CDDL-1.1
+    - name: jakarta.annotation:jakarta.annotation-api
+      license: EPL-2.0
+    - name: org.glassfish:jakarta.el
+      license: EPL-2.0
+    - name: org.jboss.spec.javax.annotation:jboss-annotations-api_1.2_spec
+      license: CDDL-1.1
+  excludes:
+    - name: javax.xml.bind:jsr173_api
diff --git 
a/dubbo-rpc/dubbo-rpc-api/src/test/java/org/apache/dubbo/rpc/support/RpcUtilsTest.java
 
b/dubbo-rpc/dubbo-rpc-api/src/test/java/org/apache/dubbo/rpc/support/RpcUtilsTest.java
index a6748a74d8..f93d896c22 100644
--- 
a/dubbo-rpc/dubbo-rpc-api/src/test/java/org/apache/dubbo/rpc/support/RpcUtilsTest.java
+++ 
b/dubbo-rpc/dubbo-rpc-api/src/test/java/org/apache/dubbo/rpc/support/RpcUtilsTest.java
@@ -370,8 +370,8 @@ public class RpcUtilsTest {
         Assertions.assertNotNull(parameterTypes5);
         Assertions.assertEquals(3, parameterTypes5.length);
         Assertions.assertEquals(String.class, parameterTypes5[0]);
-        Assertions.assertEquals(String.class, parameterTypes5[1]);
-        Assertions.assertEquals(String.class, parameterTypes5[2]);
+        Assertions.assertEquals(void.class, parameterTypes5[1]);
+        Assertions.assertEquals(Object.class, parameterTypes5[2]);
     }
 
     @ParameterizedTest

Reply via email to