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
It was discovered here that unexpected things can happen when the git-commit step's path field points to a directory within a working tree instead of to the root of the working tree.
I think the manner in which git-commit was used here was sensible, so what I propose is:
When loading an existing working tree from a user-specified path, use git rev-parse --show-toplevel to find the root of the repo/working tree and load from there.
There is a rather limited number of methods defined by the WorkTree interface that could benefit from gaining the option to work within a path that isn't the root of the working tree. They could gain the option to specify a working directory (that is verifiably within the working tree).
Between these two changes, a step like git-commit, when configured with a path that isn't the root of the working tree, would still load the working tree properly and the specified path could be used as the working directory for relevant commands.
The text was updated successfully, but these errors were encountered:
It was discovered here that unexpected things can happen when the
git-commit
step'spath
field points to a directory within a working tree instead of to the root of the working tree.I think the manner in which
git-commit
was used here was sensible, so what I propose is:When loading an existing working tree from a user-specified path, use
git rev-parse --show-toplevel
to find the root of the repo/working tree and load from there.There is a rather limited number of methods defined by the
WorkTree
interface that could benefit from gaining the option to work within a path that isn't the root of the working tree. They could gain the option to specify a working directory (that is verifiably within the working tree).Between these two changes, a step like
git-commit
, when configured with a path that isn't the root of the working tree, would still load the working tree properly and the specified path could be used as the working directory for relevant commands.The text was updated successfully, but these errors were encountered: