Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Allow changing refs for remote #363

Merged
merged 3 commits into from
Nov 15, 2022

Conversation

mrexox
Copy link
Member

@mrexox mrexox commented Nov 15, 2022

Closes #358

🔧 Summary

  • Support changing ref value (but not removing it, see below)

⚠️ There is still a problem when you had a ref, then removed it and ran lefthook install. The reason for that is - we don't know which branch to use as a default. So, the suggestion is: decide whether you use ref or not, and if you added a ref, please keep it there.

Signed-off-by: Valentin Kiselev <[email protected]>
@mrexox mrexox marked this pull request as ready for review November 15, 2022 08:17
Signed-off-by: Valentin Kiselev <[email protected]>
Signed-off-by: Valentin Kiselev <[email protected]>
@mrexox mrexox changed the title Allow changing refs for remote fix: Allow changing refs for remote Nov 15, 2022
@mrexox mrexox merged commit 2a376a7 into evilmartians:master Nov 15, 2022
@mrexox mrexox deleted the fix/allow-changing-refs-for-remote branch November 15, 2022 08:33
mrexox added a commit that referenced this pull request Nov 17, 2022
- Remove quoting for scripts (#371)
- remove lefthook.checksum on uninstall (#370)
- Print prepare-commit-msg hook if it exists in config (#368)
- Allow changing refs for remote (#363)

Signed-off-by: Valentin Kiselev <[email protected]>
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.

None yet

1 participant