Skip to content

Conversation

@th0was14
Copy link

Issue Number / Link: (If this is a minor change, then an issue does not need to be created and this line can be deleted.)
#67

@th0was14 th0was14 requested a review from hansu as a code owner October 30, 2025 20:57
"name": "git-graph-2",
"displayName": "Git Graph 2",
"version": "1.31.6",
"version": "1.31.6-alpha-1",
Copy link
Author

Choose a reason for hiding this comment

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

I can upgrade the version if you want.

Copy link
Owner

Choose a reason for hiding this comment

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

No need for an extra version

@LeuciRemi
Copy link

Oh dear, you were quicker than me with that MR. 😅
I haven't looked at it in detail but we did the same thing. 👍

@hansu
Copy link
Owner

hansu commented Oct 31, 2025

@LeuciRemi can you verify this PR? I never had to use that option...

@LeuciRemi
Copy link

I noticed a few areas that could be extended to make this feature more complete:

Missing operations:

  • pullBranch when using squash mode
  • merge when using squash mode
  • squashCommits operation

These operations all create commits and should logically support bypassing hooks, especially since users often need --no-verify when dealing with squash operations.

Implementation suggestions:

  1. The checkboxes should be grouped together to make it look more natural visually.
image
  1. Adding JSDoc comments explaining what --no-verify bypasses for each operation.

@th0was14
Copy link
Author

th0was14 commented Nov 2, 2025

@hansu do I let you take the new change ? or do you want me to do the new change ?

@hansu
Copy link
Owner

hansu commented Nov 3, 2025

@th0was14 It would be nice if you can do that :)

@th0was14
Copy link
Author

th0was14 commented Nov 8, 2025

@th0was14 It would be nice if you can do that :)

@hansu I will try. :) I let you know if I have trouble.

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.

3 participants