-
Notifications
You must be signed in to change notification settings - Fork 28
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
Add windows support for git-fastclone #65
Conversation
Created a new function for the path generation that validates the current operation system before generating the path, and uses underscore in-case a windows system is being used to make sure git-fastclone doesn't use a non-allowed characters for windows filenames that prevents the submodules and lock files from being created
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two minor pieces of feedback, but happy to land after these changes, or happy to discuss if you have pushback.
Co-authored-by: Justin Martin <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks for the contribution!
It looks like there are some minor rubocop violations to take care of. Once that's done, I'll merge this PR. Would you like me to cut a bugfix release with the changes? |
I can see where it fails, should combining the if statement as a one liner solve the issue? Something like this RbConfig::CONFIG['host_os'] =~ /mswin|msys|mingw|cygwin|bccwin|wince|emc/ ? '__' : ':' Robucop didn't say anything about the code style when I was modifying git-fastclone, I think it's better if you correct those code style violations on your side if you don't mind, I can't really validate the code style locally |
The rubocop fix looks like this:
It should produce this for you automatically if you run |
Ok, git-fastclone 1.4.5 has been published with this fix. |
Created a new function for the path generation that validates the current operation system before generating the path, and uses underscore in-case a windows system is being used to make sure git-fastclone doesn't use a non-allowed characters for windows filenames that prevents the submodules and lock files from being created