Add license check for move, typescript, javascript files (#1051)

MystenLabs · Mar 24, 2022 · 95ed0a6 · 95ed0a6 · github-actions · Mar 24, 2022
1 parent c8915bd
commit 95ed0a6
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 7 deletions.
diff --git a/explorer/client/.eslintrc.js b/explorer/client/.eslintrc.js
@@ -1,3 +1,6 @@
+// Copyright (c) 2022, Mysten Labs, Inc.
+// SPDX-License-Identifier: Apache-2.0
+
 module.exports = {
     extends: ['react-app', 'react-app/jest', 'prettier'],
     rules: {

diff --git a/explorer/client/tailwind.config.js b/explorer/client/tailwind.config.js
@@ -1,3 +1,6 @@
+// Copyright (c) 2022, Mysten Labs, Inc.
+// SPDX-License-Identifier: Apache-2.0
+
 const defaultTheme = require('tailwindcss/defaultTheme');
 
 module.exports = {

diff --git a/nft_mirror/oracle_server/src/sdk/gateway-generated-schema.ts b/nft_mirror/oracle_server/src/sdk/gateway-generated-schema.ts
@@ -1,3 +1,6 @@
+// Copyright (c) 2022, Mysten Labs, Inc.
+// SPDX-License-Identifier: Apache-2.0
+
 /**
  * This file was auto-generated by openapi-typescript.
  * Do not make direct changes to the file.

diff --git a/scripts/license_check.sh b/scripts/license_check.sh
@@ -1,20 +1,21 @@
 #!/bin/bash
-# shellcheck disable=SC2044
+# shellcheck disable=SC2044,SC2086,SC2016
 # This script checks each file starts with a license comment
 set -e
 set -o pipefail
 
 DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
 TOPLEVEL="${DIR}/../"
 
-# Iterate over rust files not in the target directory
-for i in $(find "$TOPLEVEL" -path "$TOPLEVEL/target" -prune -o -iname "*.rs" -print) 
+# Iterate over files in the repo that satisfy the following rules
+# 1. File extension is one of .(move | rs | tsx | ts | js)
+# 2. File directory is not '$TOPLEVEL/target' or "**/build" or "**/node_modules"
+for i in $(find $TOPLEVEL  -type d \( -path '$TOPLEVEL/target' -o -name 'node_modules' -o -name 'build' \) -prune -o \( -iname '*.rs' -o -iname '*.move' -o -iname '*.tsx' -o -iname '*.ts' -o -iname '*.js' \) -print)
 do
-  CNT=$(head -n3 "$i" | grep -oEe '// (Copyright \(c\) 2022, Mysten Labs, Inc.|SPDX-License-Identifier: Apache-2.0)' | wc -l)
-  # echo "$i $CNT"
+  CNT=$(head -n3 "$i" | grep -oEe '// (Copyright \(c\) 2022, Mysten Labs, Inc.|SPDX-License-Identifier: Apache-2.0)' | wc -l) || true
   if [ "$CNT" -lt 2 ]
   then
-     echo "File $i has an incorrect license header"
-     exit 1
+    echo "File $i has an incorrect license header"
+    exit 1
   fi
 done
diff --git a/sui_programmability/examples/defi/tests/EscrowTests.move b/sui_programmability/examples/defi/tests/EscrowTests.move
@@ -1,3 +1,6 @@
+// Copyright (c) 2022, Mysten Labs, Inc.
+// SPDX-License-Identifier: Apache-2.0
+
 #[test_only]
 module DeFi::EscrowTests {
     use Sui::ID::{Self, VersionedID};