Skip to content

Fix test faliure on Windows#49

Merged
fisker merged 6 commits into
mainfrom
windows
Apr 23, 2025
Merged

Fix test faliure on Windows#49
fisker merged 6 commits into
mainfrom
windows

Conversation

@fisker
Copy link
Copy Markdown
Member

@fisker fisker commented Feb 25, 2025

Fixes #47

@fisker fisker changed the title Test on windows Fix test faliure on Windows Feb 25, 2025
@fisker
Copy link
Copy Markdown
Member Author

fisker commented Feb 25, 2025

Blocked by fabiospampinato/tiny-readdir-glob#5

@fabiospampinato fabiospampinato force-pushed the main branch 4 times, most recently from 34e0257 to f8dc396 Compare March 6, 2025 00:45
@fabiospampinato
Copy link
Copy Markdown
Collaborator

It may be good to split this into two, one that enables CI for windows and linux and another that fixes the problem found by that.

@pralkarz
Copy link
Copy Markdown
Contributor

The fix for fabiospampinato/tiny-readdir-glob#5 has been released in 1.23.2.

@fabiospampinato
Copy link
Copy Markdown
Collaborator

If possible I'd split this into 2 PRs, one that sets up CI and one that fixes the issues for Windows

@fisker
Copy link
Copy Markdown
Member Author

fisker commented Apr 22, 2025

I don't understand why but here it is #56

@fisker fisker marked this pull request as ready for review April 22, 2025 15:33
@fabiospampinato
Copy link
Copy Markdown
Collaborator

It seemed cleaner, maybe also one see the the failing tests this way.

Can you rebase?

@fisker fisker merged commit a49b049 into main Apr 23, 2025
4 checks passed
@fisker fisker deleted the windows branch April 23, 2025 18:17
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.

Fix test faliure on Windows

3 participants