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.

Sign up for GitHub

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

Jump to bottom

"Listen" to conda recommendation to use some other solver upon conda install failing; stderr output from "conda install" would be captured and logged #210

Merged
merged 7 commits into from
Nov 7, 2024
Merged

"Listen" to conda recommendation to use some other solver upon conda install failing; stderr output from "conda install" would be captured and logged #210

merged 7 commits into from
Nov 7, 2024

Commits on Nov 7, 2024

  1. ci: add CI run to test currently failing installation scenario 

    ref: #206

#209 seems could provide a workaround
but I think we should test/workaround within datalad-installer without requiring adjustment
of each similar invocation
    @yarikoptic
    yarikoptic committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    843a9e8 View commit details
    Browse the repository at this point in the history

  2. fix(conda): parse and provide offered solver into "conda install" inv… 

    …ocation
    @yarikoptic
    yarikoptic committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    2ae1426 View commit details
    Browse the repository at this point in the history

  3. rf: remove no longer needed (3.7 support was dropped) conditional

    @yarikoptic
    yarikoptic committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    2018311 View commit details
    Browse the repository at this point in the history

  4. rf: move logic into a helper function to declutter the code

    @yarikoptic
    yarikoptic committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    93a6fc2 View commit details
    Browse the repository at this point in the history

  5. test: extend existing unit test to test the fix

    @yarikoptic
    yarikoptic committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    5486904 View commit details
    Browse the repository at this point in the history

  6. rf: remove dedicated scenarios workflow

    @yarikoptic
    yarikoptic committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    8c171cb View commit details
    Browse the repository at this point in the history

  7. rf: refactor test into parametric and test added on linux only 

    rf also to avoid duplication -- impossible to discern what is the difference
between two tests without doing diff, so best to have it parametric
    @yarikoptic
    yarikoptic committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    fe86e04 View commit details
    Browse the repository at this point in the history