Skip to content

update git2-hooks to 0.3.3 to use default login shell instead just bash - #4749

Merged
mtsgrd merged 1 commit into
gitbutlerapp:masterfrom
yerke:yerke/update-git2-hooks-version
Aug 25, 2024
Merged

update git2-hooks to 0.3.3 to use default login shell instead just bash#4749
mtsgrd merged 1 commit into
gitbutlerapp:masterfrom
yerke:yerke/update-git2-hooks-version

Conversation

@yerke

@yerke yerke commented Aug 24, 2024

Copy link
Copy Markdown
Contributor

☕️ Reasoning

Fixes #4569

🧢 Changes

Update git2-hooks to 0.3.3. That version includes gitui-org/gitui#2343.

🎫 Affected issues

Fixes: 4569

@vercel

vercel Bot commented Aug 24, 2024

Copy link
Copy Markdown

@yerke is attempting to deploy a commit to the GitButler Team on Vercel.

A member of the Team first needs to authorize it.

Comment thread Cargo.lock Outdated
checksum = "da03fa3b94cc19e3ebfc88c4229c49d8f08cdbd1228870a45f0ffdf84988e14b"
dependencies = [
"dirs 5.0.1",
"dirs 4.0.0",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, this downgrade looks a bit weird? gitui depends on 5.0.

@yerke yerke Aug 24, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, I am not sure why cargo decided to resolve to that version when I ran cargo update git2-hooks. shellexpand has dirs = { version = ">=4, <6", optional = true }, and dirs 4 in Cargo.lock comes from resolve-path.
Removed this change.

@yerke
yerke force-pushed the yerke/update-git2-hooks-version branch from 117331e to 5807e1d Compare August 24, 2024 18:09
@yerke
yerke requested a review from mtsgrd August 24, 2024 18:09

@krlvi krlvi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for fixing this upstream in git2_hooks <3

@mtsgrd
mtsgrd merged commit 417a024 into gitbutlerapp:master Aug 25, 2024
@yerke
yerke deleted the yerke/update-git2-hooks-version branch August 26, 2024 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use default login shell when running git-hooks

3 participants