ruby / ruby/irb

Deprecate multi-irb feature

Đang mở
#653 10 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

enhancement
Ngôn ngữ chính
Ruby
Star
478
Fork
158
Merge trung bình
1 ngày 12 giờ
Pull request đã merge (30 ngày)
12

Mô tả

Plan

  • Print deprecation warnings on these commands: irb, jobs, fg, and kill in the next minor release
  • Completely remove those commands in v2.0.0

Reasons

  • It's concept highly overlaps with workspace commands like chws, pushws...etc.
  • It spawns a new thread for every new new session, which conflicts with the upcoming irb-debug integration
  • But most importantly, we can't find any practical use for this feature and the cost of maintaining its compatibility with other features is high
    • For example, it will require the solution of #650 to support multi-thread irb sessions too

Feedback wanted

If you use the multi-irb feature and don't want to see it deprecated from IRB, please leave a comment below to explain your use case of it. It will help us assess this plan.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

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.

Hướng nghiên cứu

Bắt đầu bằng cách xác định các phần triển khai và xử lý lệnh cho irb, jobs, fgkill, sau đó kiểm tra cách các phiên bản và nội dung ngừng hỗ trợ của IRB được quản lý. Công việc được xem là hoàn tất khi các lệnh đó cảnh báo trong bản phát hành minor tiếp theo và bị xóa trong v2.0.0, đồng thời các bài kiểm thử liên quan được cập nhật.

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

Đánh giá

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

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.