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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃┕ fix: Disambiguate Windows OS Separator #198

Merged
merged 1 commit into from Jan 20, 2024
Merged

Conversation

Aagam41
Copy link
Owner

@Aagam41 Aagam41 commented Jan 20, 2024

Description

Windows uses '' (back-slash) rather than '/' (forward-slash) in its filepaths. This commit resolves #29 by replacing to correct slash based on the operating system

Motivation and Context

Types of changes

  • [x ] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • [] I have added tests to cover my changes.
  • All new and existing tests passed.

Resolves #29

Windows uses '\' (back-slash) rather than '/' (forward-slash) in its filepaths. This commit resolves this issue by replacing to correct slash on the operating system.

Resolves #29
@Aagam41 Aagam41 self-assigned this Jan 20, 2024
@Aagam41 Aagam41 merged commit 43c36f9 into master Jan 20, 2024
@Aagam41 Aagam41 deleted the dependency/django-5 branch January 20, 2024 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Windows path fix: should do like Django, not like Windows
1 participant