Skip to content
This repository has been archived by the owner on Apr 5, 2021. It is now read-only.

Changes paths to match Windows standards. #18

Merged
merged 1 commit into from
Feb 4, 2017
Merged

Changes paths to match Windows standards. #18

merged 1 commit into from
Feb 4, 2017

Conversation

MicahZoltu
Copy link
Contributor

Fixes #16 and #17.

This changes the location of several folders to match Windows standards. The full list can be found at https://msdn.microsoft.com/en-us/library/windows/desktop/dd378457(v=vs.85).aspx, though you will have to go to other MSDN articles to get details of their intended use:
Local App Data
Roaming App Data
Program Data

Note: This is a breaking change so if this project follows semantic versioning it should be released as a major version bump.

@MicahZoltu
Copy link
Contributor Author

Oops, I should have checked existing PRs first, this partially mirrors #15. I'll leave my feedback on that issue there and if it is merged I would be happy to rebase this PR.

@andybarron
Copy link
Owner

Merged #15, rebase if you please 😄

Fixes #16 and #17.

This changes the location of several folders to match Windows standards.  The full list can be found at https://msdn.microsoft.com/en-us/library/windows/desktop/dd378457(v=vs.85).aspx, though you will have to go to other MSDN articles to get details of their intended use:
[Local App Data](https://msdn.microsoft.com/en-us/windows/uwp/app-settings/store-and-retrieve-app-data#local-app-data)
[Roaming App Data](https://msdn.microsoft.com/en-us/windows/uwp/app-settings/store-and-retrieve-app-data#roaming-data)
[Program Data](https://msdn.microsoft.com/en-us/library/mt674655%28v=vs.85%29.aspx#10._apps_must_install_to_the_correct_folders_by_default)

Note: This is a breaking change so if this project follows semantic versioning it should be released as a major version bump.
@MicahZoltu
Copy link
Contributor Author

Rebased. Thanks for maintaining this project!

@andybarron
Copy link
Owner

My pleasure 👌 Merging now!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants