MetBench run prints an error and keeps going

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

Maintainer thường phản hồi trong vòng 1 ngày

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

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
84/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
go
Lĩnh vực
cli

Hướng nghiên cứu

Bắt đầu với cmd/metbench/main.go và theo dõi cách chương trình xử lý hai lỗi xác thực: n nhỏ hơn 1 và lỗi từ bench.LoadTasks(*taskDir). Trả về các mã trạng thái được yêu cầu (2 khi n không hợp lệ và 1 khi tải tác vụ thất bại), sau đó bổ sung các kiểm thử tập trung trong một main_test.go mới nếu khả thi. Hoàn thành khi cả hai lỗi đều dừng quá trình chạy với mã thoát được chỉ định.

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

Mô tả

bug good first issue

What

Two validation failures print to stderr but don't return, so the run continues with bad input. Need to return out.

Where

cmd/metbench/main.go
Possibility to add a main_test.go as well.
Two cases

  • if *n < 1
  • tasks, err := bench.LoadTasks(*taskDir)

Fix

If the user puts n as 0 return 2 for the flag error (similar to the -models) and return 1 for the load error. Worth adding a test in a new main_test.go file.

Ngôn ngữ chính
Go
Star
1
Fork
2
Merge trung bình
1 ngày 11 giờ
Pull request đã merge (30 ngày)
14

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 Edwin-metcalf/MetCode

Tất cả issue của Edwin-metcalf/MetCode

Issue tương tự

Thêm issue về Go

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.