Consolidate `else`/`if` conditional logic
@sadielbartholomew đang làm issue này rồi.
Từ ngày 10/4/2024.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
There are lots of locations in the code I have noticed where the conditional if/else logic can be streamlined or improved, so it flows more intuitively and reduces the need for extra levels of indentation, etc.
This is both in the sense of direct small-scale conversions, for example, there is a whole similar block that can be reduced with elif statements here:
as well as larger-scale changes to the conditional structure. All of which should not change the behaviour of the code in any way, only its implementation, to make it easier to work with and develop on top of.
- Ngôn ngữ chính
- Python
- Star
- 21
- Fork
- 11
- Merge trung bình
- 7 giờ 11 phút
- Pull request đã merge (30 ngày)
- 1
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của NCAS-CMS/cf-plot
-
testing
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 58/100
-
Add `AGENTS.md` fileĐang mởdocs enhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 64/100
-
Add animation hooks into cf-plotCó thể làm lại được @bnlawrence đã nhận 99 ngày trước và không có pull request nào đang mở. Đang mởenhancement
-
Ugly nemo plotting issueĐang mởbug v35plus
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
Tất cả issue của NCAS-CMS/cf-plot
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
RedHatQE/mtv-api-tests#721 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 85/100
pytest-dev/pluggy#757 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 85/100
NousResearch/hermes-agent#134960 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
HTML backend: `<br>` leaks the internal sentinel U+E000 into list items, headings and captionsCó thể đã có người làm @morten-lagabote đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
docling-project/docling#4671 ·
Maintainer thường phản hồi trong vòng 1 ngày