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

Issue warning and then ignore satisified_skip_solve on 32-bit windows #246

Merged

Conversation

mkitti
Copy link
Member

@mkitti mkitti commented Jul 19, 2023

Since conda has stopped being updated for 32-bit Windows and does not have --satisfied-skip-solve, issue a warning that it is not supported on that platform and then ignore the keyword on Conda.add.

@codecov-commenter
Copy link

codecov-commenter commented Jul 19, 2023

Codecov Report

Patch coverage: 50.00% and project coverage change: -0.43 ⚠️

Comparison is base (fe6c94c) 87.86% compared to head (058f7a2) 87.43%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #246      +/-   ##
==========================================
- Coverage   87.86%   87.43%   -0.43%     
==========================================
  Files           1        1              
  Lines         206      207       +1     
==========================================
  Hits          181      181              
- Misses         25       26       +1     
Impacted Files Coverage Δ
src/Conda.jl 87.43% <50.00%> (-0.43%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@stevengj
Copy link
Member

LGTM.

@mkitti
Copy link
Member Author

mkitti commented Jul 19, 2023

Merging expeditiously to resolve breakage then

@mkitti mkitti merged commit e1c3b61 into JuliaPy:master Jul 19, 2023
9 checks passed
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.

None yet

3 participants