Closed yarikoptic closed 4 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 65.53%. Comparing base (
d35969d
) to head (5d70ec7
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I think that does it - all checks, including macos-latest, passed.
For additional verification, I tried this in a stripped-out test run on macos-latest (i.e. macos-14-arm64) in a throwaway repo: install the installer from this branch with pip, install git-annex with the installer, install DataLad with pip, and, instead of tests, only run which
and ask about versions. It gave me the following, suggesting everything lands correctly on PATH:
/Users/runner/hostedtoolcache/Python/3.11.9/arm64/bin/datalad
/opt/homebrew/bin/git
/opt/homebrew/bin/git-annex
datalad 1.0.2
git version 2.44.0
git-annex version: 10.20240227
I'd say it's good to go.
dropped TEMP commit, force pushed, adjusted title to reflect included changes, marking for release.
@yarikoptic the release action crashed - what on earth happened there? https://github.com/datalad/datalad-installer/actions/runs/8834788909/job/24257350453
edit: ok, I think I know. For some reason the release workflow downloads an old version of auto; the issue manifested recently (some change on GitHub side?) and the latest (actually two patches back) release addressed it... Will open an issue with more details or go straight to PR.
merged my "fix" #198 , waiting on result
I also just realized that the version was simply pinned - I'll keep my fingers crossed :crossed_fingers:
FWIW -- 1.0.5 was just released so all is good. I am rerunning brew job I had https://github.com/datalad/datalad/actions/runs/8820905835?pr=7586 in https://github.com/datalad/datalad/pull/7586 which if passes is IMHO worth merging to get us 1 step closer there
expecting that macos-latest testing would not succeed. We might want to add explicitly
macos-12
and then newmacos-latest
which would be arm