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

RF: just pip install datalad-installer #5676

Merged
merged 6 commits into from May 25, 2021
Merged

Conversation

yarikoptic
Copy link
Member

master branch has its own ideas and binds to some older version

$> git grep datalad.installer
.appveyor.yml:  # deploy the datalad installer, override version via DATALAD_INSTALLER_VERSION
.appveyor.yml:      python -m pip install "datalad-installer%DATALAD_INSTALLER_VERSION%"
.appveyor.yml:      python -m pip install datalad-installer
.appveyor.yml:  - sh: python -m pip install datalad-installer${DATALAD_INSTALLER_VERSION:-}

needs yet to be addressed, decided to just go with maint for now and simplify (not versioned depends etc)

@mih
Copy link
Member

mih commented May 20, 2021

Just commenting, have no checked yet. It would be good to know if a diff to the extension setup exists, and if so why, and if all extensions need to be modified.

@mih
Copy link
Member

mih commented May 20, 2021

It is at least missing the ability to switch the installer version without editing .appveyor.yml. The feature was introduced in the extension setup the last time the installer behavior broke, and all extensions had to be fix before any CI would work again.

install:
...
  # deploy the datalad installer, override version via DATALAD_INSTALLER_VERSION
  - cmd:
      IF DEFINED DATALAD_INSTALLER_VERSION (
      python -m pip install "datalad-installer%DATALAD_INSTALLER_VERSION%"
      ) ELSE (
      python -m pip install datalad-installer
      )
  - sh: python -m pip install datalad-installer${DATALAD_INSTALLER_VERSION:-}

it would be great if the setups converge.

the only not applied change is removal of datalad.plugin testing -- still
in maint
@yarikoptic
Copy link
Member Author

it would be great if the setups converge.

it would be great if the setup didn't diverge ;-) copied master's .appveyor with only slight tune up for maint, and then improved upon. So should be re-converging now

@codecov
Copy link

codecov bot commented May 21, 2021

Codecov Report

Merging #5676 (57a66b7) into maint (0c6d355) will decrease coverage by 60.82%.
The diff coverage is n/a.

❗ Current head 57a66b7 differs from pull request most recent head c628af3. Consider uploading reports for the commit c628af3 to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##            maint    #5676       +/-   ##
===========================================
- Coverage   90.32%   29.49%   -60.83%     
===========================================
  Files         299      296        -3     
  Lines       42349    42313       -36     
===========================================
- Hits        38251    12480    -25771     
- Misses       4098    29833    +25735     
Impacted Files Coverage Δ
datalad/tests/test_api.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/support/digests.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/tests/test_base.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/tests/test_config.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/ui/tests/__init__.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/tests/test__main__.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/tests/test_strings.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/ui/tests/test_base.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/support/tests/utils.py 0.00% <0.00%> (-100.00%) ⬇️
datalad/tests/test_testrepos.py 0.00% <0.00%> (-100.00%) ⬇️
... and 246 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0c6d355...c628af3. Read the comment docs.

@yarikoptic
Copy link
Member Author

hm, codecov offset is too big. #5688 against maint has only 0.1% . looking at diff in coverage - might relate to travis runs for high log level to not provide results, but also might relate to added INSTALL_GITANNEX: git-annex -m datalad/packages. so I will comment it out and give another full round to see if codecov report comes cleaner.

@yarikoptic
Copy link
Member Author

ok, had to restart one appveyor on mac. now all green and codecov divergence is small. so we should be good to go

@yarikoptic yarikoptic added the merge-if-ok OP considers this work done, and requests review/merge label May 24, 2021
Copy link
Member

@mih mih left a comment

Choose a reason for hiding this comment

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

Wonderful, thx @yarikoptic for the updates!

@mih mih merged commit 6222595 into datalad:maint May 25, 2021
@yarikoptic yarikoptic added the semver-patch Increment the patch version when merged label Jun 2, 2021
@yarikoptic yarikoptic mentioned this pull request Jun 3, 2021
8 tasks
@yarikoptic yarikoptic deleted the rf-installer branch October 8, 2021 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-if-ok OP considers this work done, and requests review/merge semver-patch Increment the patch version when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants