Change Missing From Changelog: forEach() Loop's `Return` returns to forEach()'s invocation point, not to the calling function.
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- JavaScript
- Star
- 11.8k
- Fork
- 7.9k
- Merge trung bình
- 1 ngày 11 giờ
- Pull request đã merge (30 ngày)
- 11
Mô tả
Hey all,
I'm noticing that in upgrading to ReactJS Version 16.12.0 from 16.7.0, the behavior of foreach was changed, so that return returns to where foreach is called, and not the function calling the function containing the foreach. But I don't see this change here:
https://github.com/facebook/react/blob/master/CHANGELOG.md
Basically,
function1() {
function2();
console.log("function 1");
}
function2() {
const arr = ['cool'];
arr.forEach((var) => {
return var;
});
console.log("function 2");
}
Before 16.12.0, console output was: "function 1", but now it is "function 2; function 1." This change appears to be against MDN documentation and internet consensus: https://stackoverflow.com/questions/34653612/what-does-return-keyword-mean-inside-foreach-function
But it should have been included in the changelog, though I am not seeing it. Let me know if I'm misunderstanding.
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 với CHANGELOG.md và mục changelog của React 16.12.0 được issue tham chiếu. Xác minh hành vi forEach() được báo cáo dựa trên các tham chiếu MDN và Stack Overflow được liên kết, sau đó xác định xem changelog có thiếu mục nào không; được xem là hoàn thành khi báo cáo đã được xác thực và changelog phản ánh chính xác bản release nếu phù hợ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ệ
- javascript, react
- Lĩnh vực
- documentation
- Loại issue
- Tài liệu
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- 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
- 30/100