imbajin commented on code in PR #197:
URL:
https://github.com/apache/incubator-hugegraph-doc/pull/197#discussion_r1103645444
##########
.github/workflows/validate-release.yml:
##########
@@ -104,27 +109,34 @@ jobs:
if [[ ! -f "DISCLAIMER" ]]; then
echo "The package should include DISCLAIMER file" && exit 1
fi
+ # 4.2 ensure doesn't contains
*GPL/BCL/JSR-275/RSAL/QPL/SSPL/CPOL/NPL1.*/CC-BY
+ # dependency in LICENSE and NOTICE file
+ COUNT=$(grep -E "GPL|BCL|JSR-275|RSAL|QPL|SSPL|CPOL|NPL1|CC-BY"
LICENSE NOTICE | wc -l)
+ if [[ $COUNT -ne 0 ]]; then
+ grep -E "GPL|BCL|JSR-275|RSAL|QPL|SSPL|CPOL|NPL1.0|CC-BY"
LICENSE NOTICE
+ echo "The package shouldn't include GPL* invalid dependency, but
get $COUNT" && exit 1
+ fi
- # 4.2 ensure doesn't contains empty directory or file
+ # 4.3 ensure doesn't contains empty directory or file
COUNT=$(find . -type d -empty | wc -l)
if [[ $COUNT -ne 0 ]]; then
find . -type d -empty
- echo "The package shouldn't include empty directory, but get
$COUNT" # TODO: && exit 1
+ echo "The package shouldn't include empty directory, but get
$COUNT" && exit 1
fi
- # 4.3 ensure any file should less than 900kb & not include binary
file
+ # 4.4 ensure any file should less than 900kb & not include binary
file
COUNT=$(find . -type f -size +900k | wc -l)
if [[ $COUNT -ne 0 ]]; then
find . -type f -size +900k
- echo "The package shouldn't include file larger than 900kb, but
get $COUNT"
+ echo "The package shouldn't include file larger than 900kb, but
get $COUNT" && exit 1
fi
COUNT=$(find . -type f | perl -lne 'print if -B' | grep -v *.txt |
wc -l)
Review Comment:
need a better way to search binary files（and exclude some files marked in
properties）
no we can't exit here，but we should search the key word in action log
manually
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@hugegraph.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
