Feature: check formatting before push #838
Conversation
fairax
force-pushed
the
feature/pre-push-checks
branch
from
a7da4e8 to
73e5a49
Compare
init.sh
Outdated
| @@ -0,0 +1 @@ | |||
| git config --local core.hooksPath ./.githooks | |||
There was a problem hiding this comment.
Скопируй хук в .git, чтобы изменениями репозитория нельзя было вызвать RCE
There was a problem hiding this comment.
Лучше вообще в README команду перенести, а то init.sh звучит слишком generic)
Либо в мейкфайл, который у нас в качестве task runner используется
There was a problem hiding this comment.
Может расширить init.sh? Добавить туда
cd tests
yarn install
?
There was a problem hiding this comment.
Да всё равно лучше на таск-раннер это спихнуть, не надо случайными скриптами корень репы забивать
There was a problem hiding this comment.
Скопируй хук в .git, чтобы изменениями репозитория нельзя было вызвать RCE
Ну во-первых, это в любом случае не RCE, т.к. вызов кода происходит не удалённо, а только если пользователь сам случайно вызовет вредоносный код.
Во-вторых, если кто-то может положить вредоносный код в репозиторий, обойдя code review, то у него есть ещё куча способов навредить и без изменения этого конкретного хука.
Вместе с тем при переносе кода в .git пользователям придётся вызывать скрипт копирования каждый раз когда мы что-то меняем в хуке. Сообщить пользователю о необходимости вызвать скрипт копирования хотябы один раз уже проблема, а сообщать ему каким-то образом о том, что нужно вызвать скрипт повторно, это вообще по моему не реально.
fairax
force-pushed
the
feature/pre-push-checks
branch
2 times, most recently
from
cf2437d to
96ddf6c
Compare
fairax
force-pushed
the
feature/pre-push-checks
branch
3 times, most recently
from
9e008a7 to
63c44f4
Compare
README.md
Outdated
| @@ -159,6 +159,11 @@ pushd tests && yarn fix ; popd | |||
| cd tests && yarn eslint --ext .ts,.js src/ | |||
| ``` | |||
|
|
|||
| ### Enable checking of code style on commits | |||
| ```bash | |||
| make init-fmt-on-commit | |||
node_modules/.yarn-integrity
Outdated
| @@ -0,0 +1,10 @@ | |||
| { | |||
There was a problem hiding this comment.
Ты вызвал yarn в корневой директории, этих файлов быть не должно
fairax
force-pushed
the
feature/pre-push-checks
branch
from
63c44f4 to
cd77dae
Compare
No description provided.