OpenAPITools / OpenAPITools/openapi-diff
PUT new attribute backwards compatibility optional
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Java
- Star
- 1.1k
- Fork
- 190
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
https://github.com/OpenAPITools/openapi-diff/issues/136 made the PUT request incompatibility check stricter, considering an additional field as a breaking change. This makes sense for a scenario where updates are taking place, but is too strict if PUTs represent only inserts.
What would be the best way of optionally removing the strict handling of PUTs? Options could include:
- Surfacing more information about how the
changedOperation.isIncompatible()decision was made, allowing the user of the OpenApiDiff to use more granular information to decide whether it should be considered as a breaking change. - Adding a command-line flag to remove the PUT strictness for all cases in a run.
- Writing some code to traverse the object graph for PUTs when the diff has completed and decide whether the strict PUT handling is what caused the incompatibility, and then ignore it.
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.
Hướng nghiên cứu
Bắt đầu bằng cách theo dõi cách changedOperation.isIncompatible() xử lý các trường được thêm vào trong các yêu cầu PUT và cách kết quả đó đi đến diff đã hoàn tất. So sánh các phương án được đề xuất: cung cấp thông tin chi tiết về tính không tương thích, thêm một cờ dòng lệnh hoặc duyệt qua đồ thị đối tượng sau khi diff. Được xem là hoàn tất khi một thiết kế duy nhất đã được chọn và đặc tả, đồng thời hành vi của thiết kế đó đối với các PUT chỉ chèn và các PUT kiểu cập nhật đã được đề cập.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- java
- Lĩnh vực
- api
- Loại issue
- Tính năng
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 30/100