You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#18 provides for use cases of pre-push hooks specifically. #16 (comment) provides analysis for why a lightweight implementation is superior beyond the mere benefit of being leightweight.
That means, the consolidated intended use case does not require git workdir management. What's worse, providing such would induce misinterpretations about the consolidated intended use case.
Therefore, it is proposed for devshell to arrange for:
A simple command sequence
that is defined in devshell's standard interfaces
and that is executed in order
in the context of the repo's root and within debshell env
by a devshell library facility
which is symlinked to .git/hooks/pre-push.
The facility is capable of detecting modified files against the reference git ref,
which in the case of pre-push
is origin's HEAD.
The facility passes the modified files list
as last variadic argument
to each command in the list.
The command itself needs to arrange for
handling this variadic last argument (discarding if necessary),
filtering on file types against this variadic last argument
devshell may provide a configurable generic wrapper script for filtering, that supports configurable matching through:
#18 provides for use cases of pre-push hooks specifically. #16 (comment) provides analysis for why a lightweight implementation is superior beyond the mere benefit of being leightweight.
That means, the consolidated intended use case does not require git workdir management. What's worse, providing such would induce misinterpretations about the consolidated intended use case.
Therefore, it is proposed for devshell to arrange for:
A simple command sequence
.git/hooks/pre-push
.The facility is capable of detecting modified files against the reference git ref,
The facility passes the modified files list
The command itself needs to arrange for
identify
The text was updated successfully, but these errors were encountered: