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

Rewrite #1

Merged
merged 45 commits into from
Jan 5, 2021
Merged

Rewrite #1

merged 45 commits into from
Jan 5, 2021

Conversation

jwodder
Copy link
Member

@jwodder jwodder commented Dec 11, 2020

@codecov-io
Copy link

codecov-io commented Dec 15, 2020

Codecov Report

Merging #1 (9e011b7) into master (acd031b) will increase coverage by 32.42%.
The diff coverage is 63.83%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master       #1       +/-   ##
===========================================
+ Coverage   31.84%   64.26%   +32.42%     
===========================================
  Files           1        1               
  Lines         292      876      +584     
  Branches       47      167      +120     
===========================================
+ Hits           93      563      +470     
- Misses        181      258       +77     
- Partials       18       55       +37     
Impacted Files Coverage Δ
src/datalad_installer.py 64.26% <63.83%> (+32.42%) ⬆️

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 acd031b...9e011b7. Read the comment docs.

@jwodder jwodder marked this pull request as ready for review December 17, 2020 21:35
@jwodder
Copy link
Member Author

jwodder commented Dec 17, 2020

Note that the pypy3 failures are due to a problem with the setup-python action.

@yarikoptic
Copy link
Member

FTR that issue was fixed in 2.2.1, I've restarted the jobs

yarikoptic and others added 2 commits December 23, 2020 22:09
It already carries a shebang.  Having it executable simplifies the usecase
of having this repository cloned and just trying to use it.  I am not sure
if there are any side-effects for it being used as Python module
ENH: make datalad_installer.py script executable
@yarikoptic
Copy link
Member

It is so green and much more complete than before, so to facilitate testing let's proceed with testing it merged

@yarikoptic yarikoptic merged commit 92fd2f4 into master Jan 5, 2021
@yarikoptic yarikoptic deleted the rewrite01 branch January 5, 2021 21:31
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.

install.py: flexible/modular and easy on user (next round)
3 participants