Skip to content

Add macOS and Windows CI testing#119

Merged
yarikoptic merged 1 commit intoLykos153:masterfrom
yarikoptic:enh-test
Dec 12, 2025
Merged

Add macOS and Windows CI testing#119
yarikoptic merged 1 commit intoLykos153:masterfrom
yarikoptic:enh-test

Conversation

@yarikoptic
Copy link
Copy Markdown
Collaborator

Extend the test matrix to include macOS and Windows platforms with Python 3.9, while keeping all Python versions on Ubuntu. Use pip to install git-annex cross-platform instead of apt.

🤖 Generated with Claude Code

I have not realized that we do not test across OSes thus potentially creating an issue for usage within datalad. Also related is

TODOs:

  • make work under Windows? I forgot how the special remotes execution etc is concocted under windows, so we might need tune for testing there.

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.10%. Comparing base (c75c86c) to head (7bf4b81).
⚠️ Report is 32 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #119   +/-   ##
=======================================
  Coverage   96.10%   96.10%           
=======================================
  Files           2        2           
  Lines         385      385           
=======================================
  Hits          370      370           
  Misses         15       15           
Flag Coverage Δ
unittests 96.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Extend the test matrix to include macOS and Windows platforms with
Python 3.9, while keeping all Python versions on Ubuntu. Use pip to
install git-annex cross-platform instead of apt.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@yarikoptic
Copy link
Copy Markdown
Collaborator Author

ok, we do not try to run that bash example on windows and that makes it all kosher at least in terms of unittesting the library . I will proceed with merge

@yarikoptic yarikoptic added the tests Add or improve existing tests label Dec 12, 2025
@yarikoptic yarikoptic merged commit ad690db into Lykos153:master Dec 12, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Add or improve existing tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant