Remove executable flag from file mode (#6185)
* Remove executable flag from file mode Affected: dgraph, django, homebrew, pagekit, uikit * Add lint step to CI to reject executable icons Co-authored-by: Eric Cornelissen <ericornelissen@gmail.com>
18
.github/workflows/verify.yml
vendored
@ -44,6 +44,24 @@ jobs:
|
|||||||
run: npm ci
|
run: npm ci
|
||||||
- name: Run linter
|
- name: Run linter
|
||||||
run: npm run lint
|
run: npm run lint
|
||||||
|
- name: Verify file permissions
|
||||||
|
run: |
|
||||||
|
CHECK_DIRS="icons/ _data/"
|
||||||
|
echo "Searching the following directories for executable files:"
|
||||||
|
echo "${CHECK_DIRS}"
|
||||||
|
echo ""
|
||||||
|
EXE_FILES=$(find ${CHECK_DIRS} -type f -executable)
|
||||||
|
if test -n "${EXE_FILES-}"
|
||||||
|
then
|
||||||
|
echo "Some files were detected to have their executable bit set."
|
||||||
|
echo "To fix this, you can use 'chmod -x PATH/TO/FILE' on the following files:"
|
||||||
|
echo ""
|
||||||
|
echo "${EXE_FILES}"
|
||||||
|
exit 1
|
||||||
|
else
|
||||||
|
echo "All clear."
|
||||||
|
exit 0
|
||||||
|
fi
|
||||||
test:
|
test:
|
||||||
name: Test package
|
name: Test package
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
0
icons/dgraph.svg
Executable file → Normal file
Before Width: | Height: | Size: 821 B After Width: | Height: | Size: 821 B |
0
icons/django.svg
Executable file → Normal file
Before Width: | Height: | Size: 581 B After Width: | Height: | Size: 581 B |
0
icons/homebrew.svg
Executable file → Normal file
Before Width: | Height: | Size: 3.6 KiB After Width: | Height: | Size: 3.6 KiB |
0
icons/pagekit.svg
Executable file → Normal file
Before Width: | Height: | Size: 186 B After Width: | Height: | Size: 186 B |
0
icons/uikit.svg
Executable file → Normal file
Before Width: | Height: | Size: 295 B After Width: | Height: | Size: 295 B |