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

Feature work with husky (#47) #48

Closed
wants to merge 6 commits into from

Conversation

matt-degraffenreid
Copy link
Contributor

feat(githook.ts): add support for git core hooks path and fallback to default symlink url if not found

feat(utils/git.ts): add getCoreHooksPath function to retrieve the path of the core hooks directory.

feat(githook.ts): add support for git core hooks path and fallback to default symlink url if not found

feat(utils/git.ts): add getCoreHooksPath function to retrieve the path of the core hooks directory.
@matt-degraffenreid
Copy link
Contributor Author

Pull request for #47

@matt-degraffenreid
Copy link
Contributor Author

Noticing some undesired behavior will need another commit to fix

@di-sukharev di-sukharev changed the base branch from master to dev March 24, 2023 02:08
@di-sukharev
Copy link
Owner

changed base branch to dev

@di-sukharev
Copy link
Owner

@matt-degraffenreid bro, awesome PR, let's merge when you solve the conflicts :)

@di-sukharev
Copy link
Owner

@matt-degraffenreid bro, could you also add this fix for windows #11 (comment)

…pareCommitMessageHook function to stage all changes if flag is true
@matt-degraffenreid
Copy link
Contributor Author

Fixed the issue I had with behavior there was a flag missing from the hook process causing all files to get staged. Copy and pasted the code from the commit process to make the flag. Will take a look at number #11 and fix the merge conflicts

@matt-degraffenreid
Copy link
Contributor Author

Resolved the conflicts, and addressed #11. Just need a reviewer and this one is good to merge into dev

@di-sukharev
Copy link
Owner

di-sukharev commented Mar 29, 2023

@matt-degraffenreid bro, solve the conflicts pls and i will merge it

@matt-degraffenreid
Copy link
Contributor Author

Making a new pull request fubared this one

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

3 participants