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

Enable windows tests again, the first steps (#528) #640

Merged
merged 1 commit into from
Nov 17, 2023
Merged

Conversation

halfow
Copy link
Contributor

@halfow halfow commented Aug 8, 2023

To allow for the CI to run the windows tests again, I've done the bare minimum to get the tox test runner to run again under Windows. There are failing tests but, the purpose of this pull request is just to make it run again.

  • Resolve XDG paths in a OS independent way (using pathlib.Path to do all \ and / convertions)
  • Create directories for config file if they doesn't exist.
  • Workaround for removal of tmp XDG files on windows. shutil.rmtree sometimes fails one windows, seems to be UAC related.

Needs to run on the CI for Linux verdict just tested for python versions 3{6-11} on windows 11 😄

* Resolve XDG paths in a OS independent way (using pathlib.Path)
* Create directories for config file if it does not exist.
* Workaround for removal of tmp XDG files on windows.
  shutil.rmtree sometimes fails one windows, seems to be UAC related.
@olofk olofk merged commit b72d3ff into olofk:main Nov 17, 2023
13 checks passed
@olofk
Copy link
Owner

olofk commented Nov 17, 2023

Excellent! Thank you for working on this. Picked and pushed!

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.

2 participants