LSSTDESC / rail

Top level "umbrella" package for RAIL
MIT License
8 stars 4 forks source link

fix installation #91

Closed eacharles closed 10 months ago

eacharles commented 10 months ago

Change Description

Solution Description

Code Quality

Project-Specific Pull Request Checklists

Bug Fix Checklist

New Feature Checklist

Documentation Change Checklist

Build/CI Change Checklist

Other Change Checklist

codecov[bot] commented 10 months ago

Codecov Report

All modified lines are covered by tests :white_check_mark:

Comparison is base (e4f0d85) 100.00% compared to head (53bac20) 100.00%. Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #91 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 1 1 Lines 1 1 ========================================= Hits 1 1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

OliviaLynn commented 10 months ago

Hm, this does make me realize some weird splits in the installation requirements, between m1s, macs, and zsh:

However, I'm a little worried that making zsh the focus of the tab would detract from the conda/healpy requirement further down. Also, making General/Mac/Apple-Silicon tabs seems excessive. Maybe it would be best to just take the delight instructions out of the tab? I'd be happy to talk this over tomorrow.