Improvement: Make the CLI work on windows #161
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
| @ShreyPurohit Thanks a lot for your work on this! very happy that we're now officially supporting Windows thanks to you |
FOLKS-Tech pushed a commit to FOLKS-Tech/gitingest that referenced this pull request Sep 5, 2025
* Improvement: Make the CLI work on windows * Fix tmp file creation and add test * add error message when git missing * update CI to test windows and Macos --------- Co-authored-by: Romain Courtois <romain@coderamp.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
Issue Link: Improvement: Make the CLI work on windows
Key changes:
_normalize_pathfunction to handle platform-specific path normalization_normalize_path_strfunction to ensure consistent forward slash separators in output_get_encoding_listfunction for cross-platform file encoding detectionsymlinkhandling in_is_safe_symlink** Additionally, this project would not run on Windows Systems due to
setup.pyhandling Readme File.Fixed the above as well