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

Add Platform.normalize_path to ensure standard path strings #111

Merged
merged 1 commit into from
Nov 16, 2024

Conversation

MHendricks
Copy link
Member

WinPlatform.normalize_path ensures that drive letters are always capitalized.

This fixes issues with windows drive letters not being consistently saved in the habcache. When platform_path_map is not used on a habcache'd path the cache would not find and load the cached value.

Checklist

  • I have read the CONTRIBUTING.md document
  • I formatted my changes with black
  • I linted my changes with flake8
  • I have added documentation regarding my changes where necessary
  • Any pre-existing tests continue to pass
  • Additional tests were made covering my changes

Types of Changes

  • Bugfix (change that fixes an issue)
  • New Feature (change that adds functionality)
  • Documentation Update (if none of the other choices apply)

Proposed Changes

@MHendricks MHendricks force-pushed the mikeh/norm_case branch 2 times, most recently from 17f49fa to a077d35 Compare November 15, 2024 23:55
`WinPlatform.normalize_path` ensures that drive letters are always capitalized.

This fixes issues with windows drive letters not being consistently saved
in the habcache. When platform_path_map is not used on a habcache'd path
the cache would not find and load the cached value.
@MHendricks MHendricks merged commit a429d01 into main Nov 16, 2024
31 checks passed
@MHendricks MHendricks deleted the mikeh/norm_case branch November 16, 2024 00:16
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.

1 participant