Skip to content

Commit

Permalink
Fix license header and always run license check (apache#6222)
Browse files Browse the repository at this point in the history
Motivation

Fix the license check errors.

* Fix license header
* Always run license check
  • Loading branch information
sijie authored Feb 5, 2020
1 parent 009e03f commit a8e56b9
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 10 deletions.
1 change: 1 addition & 0 deletions .asf.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
# specific language governing permissions and limitations
# under the License.
#

github:
description: "Apache Pulsar - distributed pub-sub messaging system"
homepage: https://pulsar.apache.org/
Expand Down
10 changes: 0 additions & 10 deletions .github/workflows/ci-license.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,29 +34,19 @@ jobs:
- name: checkout
uses: actions/checkout@v1

- name: Check if this pull request only changes documentation
id: docs
uses: apache/pulsar-test-infra/diff-only@master
with:
args: site2 .github deployment .asf.yaml

- name: Set up JDK 1.8
uses: actions/setup-java@v1
if: steps.docs.outputs.changed_only == 'no'
with:
java-version: 1.8

# license check fails with 3.6.2 so we have to downgrade
- name: Set up Maven
uses: apache/pulsar-test-infra/setup-maven@master
if: steps.docs.outputs.changed_only == 'no'
with:
maven-version: 3.6.1

- name: build and check license
if: steps.docs.outputs.changed_only == 'no'
run: mvn -B -ntp -DskipTests license:check install

- name: license check
if: steps.docs.outputs.changed_only == 'no'
run: src/check-binary-license ./distribution/server/target/apache-pulsar-*-bin.tar.gz

0 comments on commit a8e56b9

Please sign in to comment.