Change Missing From Changelog: forEach() Loop's `Return` returns to forEach()'s invocation point, not to the calling function.
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 11.8k
- Forks
- 7.9k
- Ø Merge
- 1 T. 11 Std.
- Gemergte PRs (30 T.)
- 11
Beschreibung
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.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit CHANGELOG.md und dem im Issue referenzierten Changelog-Eintrag für React 16.12.0. Überprüfe das gemeldete Verhalten von forEach() anhand der verlinkten MDN- und Stack-Overflow-Referenzen und ermittle anschließend, ob im Changelog ein Eintrag fehlt; abgeschlossen ist die Aufgabe, wenn der Bericht validiert ist und der Changelog die Veröffentlichung gegebenenfalls korrekt widerspiegelt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, react
- Bereich
- documentation
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 30/100