Rewrite the code using objects and methods

Đang mở
  • Có gì không ổn?
#38 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
48/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
ci-cd

Hướng nghiên cứu

Không có đường dẫn tệp hoặc đường dẫn kiểm thử nào được nêu; hãy bắt đầu bằng cách tìm vòng lặp qua các tệp JSON và entry point count_bucketed_by_test. Theo dõi cách từng cấu trúc bucketing được khởi tạo và sửa đổi, sau đó tái cấu trúc luồng đó xoay quanh các đối tượng và phương thức, đồng thời giữ nguyên hành vi bucketing hiện có và cải thiện khả năng kiểm thử.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

The Ciclops code is conceptually simple, but syntactically muddled.
There is a for loop over all the JSON files produced by running a test suite.
Each of these files is bucketed by different criteria. E.g. by kubernetes version, or by test name.

Because it was not written in an object-based way, the code requires to pass and modify data structures by reference.
Those structures need to be initialized outside the code that uses them, like so.

    by_test = {}
    ...
    for file in dir_listing:
        ...
        with open(path, encoding="utf-8") as json_file:
            ...
            # bucketing by test name
            count_bucketed_by_test(test_results, by_test)

By rewriting the code in an object-based way, the data structures can be created and added to by methods, with more compact code. This will also improve the testability of the code.

Ngôn ngữ chính
Python
Star
16
Fork
0
Merge trung bình
6 giờ 59 phút
Pull request đã merge (30 ngày)
1

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của cloudnative-pg/ciclops

Tất cả issue của cloudnative-pg/ciclops

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.