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 lib/.gitignore path separator on Windows. #5325

Merged

Conversation

Kytech
Copy link
Contributor

@Kytech Kytech commented Sep 9, 2022

When working on Windows, the script responsible for generating lib/.gitignore when executed by git hooks uses the wrong path separator for .gitignore files, preventing TypeScript-generated JS files from being consistently ignored on Windows machines.

On Windows only, the generated path uses the Windows-style backslash path separator due to node's path package conventions. .gitignore syntax always requires use of the POSIX-style forward slash. (Reference). This simply normalizes all path separators to forward slashes in the outputted .gitignore file.

Just a very minor issue I encountered when working with the codebase that I had to fix.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.327% when pulling 27640d7 on ITDeptUtahCountyGovernment:lib-gitignore-fix into dc6dbbf on knex:master.

@OlivierCavadenti OlivierCavadenti merged commit 5c4837c into knex:master Nov 14, 2022
@Kytech Kytech deleted the lib-gitignore-fix branch October 12, 2023 16:31
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