Skip to content

Conversation

@yarikoptic
Copy link

@yarikoptic yarikoptic commented Mar 30, 2023

Would not test much if at all, but at least would try to install/import across all OSes and Python versions.

ref #1

edit: I also pushed as master to my clone so that action is executed... can be observed on https://github.com/yarikoptic/refuse/actions .

@yarikoptic yarikoptic mentioned this pull request Mar 30, 2023
# +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

import os
import sys
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

congratulate on the first bug fix contributed due to initiating the testing ;)

run: |
$url = 'https://github.com/winfsp/winfsp/releases/download/v2.0/winfsp-2.0.23075.msi'
Invoke-WebRequest -Uri $url -OutFile 'winfsp.msi'
Start-Process msiexec.exe -ArgumentList '/i', 'winfsp.msi', '/quiet', '/norestart' -Wait
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well -- either low is not intended for Windows at all or smth else need to be done since we get ERROR tests/test_low.py - OSError: Unable to find libfuse . See https://github.com/yarikoptic/refuse/actions/runs/4559603821/jobs/8043726950 for more detail

if: startsWith(matrix.os, 'macos')
run: |
brew update
brew install osxfuse
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

attempt to run tests with this seems to just stall on macos-11 without any feedback: see https://github.com/yarikoptic/refuse/actions/runs/4559603821/jobs/8043727829 (still running ,will check if outputs anything tomorrow)

@yarikoptic
Copy link
Author

both windows and OSX runs aren't happy so I will disable them for now since it annoys me to receive all the failing notifications emails ;-)

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