Skip to content

Commit

Permalink
Merge pull request #1173 from mathbunnyru/asalikhov/fix_bashate
Browse files Browse the repository at this point in the history
Fix bashate style issues
  • Loading branch information
romainx authored Sep 30, 2020
2 parents 150731d + 5b678ca commit 36d857b
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 17 deletions.
1 change: 1 addition & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ repos:
rev: 2.0.0
hooks:
- id: bashate
args: ['--ignore=E006']
- repo: https://gitlab.com/pycqa/flake8
rev: 3.8.3
hooks:
Expand Down
28 changes: 14 additions & 14 deletions base-notebook/fix-permissions
Original file line number Diff line number Diff line change
Expand Up @@ -18,18 +18,18 @@
set -e

for d in "$@"; do
find "$d" \
! \( \
-group $NB_GID \
-a -perm -g+rwX \
\) \
-exec chgrp $NB_GID {} \; \
-exec chmod g+rwX {} \;
# setuid,setgid *on directories only*
find "$d" \
\( \
-type d \
-a ! -perm -6000 \
\) \
-exec chmod +6000 {} \;
find "$d" \
! \( \
-group $NB_GID \
-a -perm -g+rwX \
\) \
-exec chgrp $NB_GID {} \; \
-exec chmod g+rwX {} \;
# setuid, setgid *on directories only*
find "$d" \
\( \
-type d \
-a ! -perm -6000 \
\) \
-exec chmod +6000 {} \;
done
2 changes: 1 addition & 1 deletion minimal-notebook/hooks/run_hook
Original file line number Diff line number Diff line change
Expand Up @@ -42,4 +42,4 @@ $(docker run --rm ${IMAGE_NAME} conda list)
\`\`\`
$(docker run --rm ${IMAGE_NAME} apt list --installed)
\`\`\`
EOF
EOF
2 changes: 1 addition & 1 deletion r-notebook/hooks/run_hook
Original file line number Diff line number Diff line change
Expand Up @@ -52,4 +52,4 @@ $(docker run --rm ${IMAGE_NAME} conda list)
\`\`\`
$(docker run --rm ${IMAGE_NAME} apt list --installed)
\`\`\`
EOF
EOF
2 changes: 1 addition & 1 deletion tensorflow-notebook/hooks/run_hook
Original file line number Diff line number Diff line change
Expand Up @@ -42,4 +42,4 @@ $(docker run --rm ${IMAGE_NAME} conda list)
\`\`\`
$(docker run --rm ${IMAGE_NAME} apt list --installed)
\`\`\`
EOF
EOF

0 comments on commit 36d857b

Please sign in to comment.