Skip to content

Commit 27c506c

Browse files
Correct the chronological order of proposal review discussions (#2402)
For these proposals, also: - Normalize discussion naming according to template instructions (e.g. 'first pitch', 'acceptance', 'returned' - Normalize use of 'Review' heading label
1 parent 4be187c commit 27c506c

6 files changed

+6
-6
lines changed

proposals/0306-actors.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
* Authors: [John McCall](https://github.com/rjmccall), [Doug Gregor](https://github.com/DougGregor), [Konrad Malawski](https://github.com/ktoso), [Chris Lattner](https://github.com/lattner)
55
* Review Manager: [Joe Groff](https://github.com/jckarter)
66
* Status: **Implemented (Swift 5.5)**
7-
* Decision Notes: [Acceptance](https://forums.swift.org/t/accepted-with-modification-se-0306-actors/47662), [First Review](https://forums.swift.org/t/se-0306-actors/45734), [Second Review](https://forums.swift.org/t/se-0306-second-review-actors/47291)
87
* Implementation: Partially available in [recent `main` snapshots](https://swift.org/download/#snapshots) behind the flag `-Xfrontend -enable-experimental-concurrency`
8+
* Review: ([first review](https://forums.swift.org/t/se-0306-actors/45734)), ([second review](https://forums.swift.org/t/se-0306-second-review-actors/47291)), ([acceptance](https://forums.swift.org/t/accepted-with-modification-se-0306-actors/47662))
99

1010
## Table of Contents
1111

proposals/0395-observability.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* Authors: [Philippe Hausler](https://github.com/phausler), [Nate Cook](https://github.com/natecook1000)
55
* Review Manager: [Ben Cohen](https://github.com/airspeedswift)
66
* Status: **Implemented (Swift 5.9)**
7-
* Review: [(pitch)](https://forums.swift.org/t/pitch-observation-revised/63757) [(first review)](https://forums.swift.org/t/se-0395-observability/64342/) [(second review)](https://forums.swift.org/t/second-review-se-0395-observability/65261/) [(acceptance)](https://forums.swift.org/t/accepted-with-revision-se-0395-observability/66760)
7+
* Review: ([pitch](https://forums.swift.org/t/pitch-observation-revised/63757)), ([first review](https://forums.swift.org/t/se-0395-observability/64342/)), ([second review](https://forums.swift.org/t/second-review-se-0395-observability/65261/)), ([acceptance](https://forums.swift.org/t/accepted-with-revision-se-0395-observability/66760))
88

99
#### Changes
1010

proposals/0411-isolated-default-values.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
* Bug: *if applicable* [apple/swift#58177](https://github.com/apple/swift/issues/58177)
88
* Implementation: [apple/swift#68794](https://github.com/apple/swift/pull/68794)
99
* Upcoming Feature Flag: `IsolatedDefaultValues`
10-
* Review: ([acceptance](https://forums.swift.org/t/accepted-se-0411-isolated-default-value-expressions/68806)) ([review](https://forums.swift.org/t/se-0411/68065)) ([pitch](https://forums.swift.org/t/pitch-isolated-default-value-expressions/67714))
10+
* Review: ([pitch](https://forums.swift.org/t/pitch-isolated-default-value-expressions/67714)), ([review](https://forums.swift.org/t/se-0411/68065)), ([acceptance](https://forums.swift.org/t/accepted-se-0411-isolated-default-value-expressions/68806))
1111

1212
## Introduction
1313

proposals/0414-region-based-isolation.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
* Review Manager: [Holly Borla](https://github.com/hborla)
66
* Status: **Accepted**
77
* Implementation: On `main` gated behind `-enable-experimental-feature RegionBasedIsolation`
8-
* Review: ([acceptance](https://forums.swift.org/t/accepted-with-modifications-se-0414-region-based-isolation/70051)), ([second review](https://forums.swift.org/t/se-0414-second-review-region-based-isolation/69740)), ([decision notes](https://forums.swift.org/t/returned-for-revision-se-0414-region-based-isolation/69123)), ([review](https://forums.swift.org/t/se-0414-region-based-isolation/68805)), ([pitch 2](https://forums.swift.org/t/pitch-region-based-isolation/67888)), ([pitch 1](https://forums.swift.org/t/pitch-safely-sending-non-sendable-values-across-isolation-domains/66566))
8+
* Review: ([first pitch](https://forums.swift.org/t/pitch-safely-sending-non-sendable-values-across-isolation-domains/66566)), ([second pitch](https://forums.swift.org/t/pitch-region-based-isolation/67888)), ([first review](https://forums.swift.org/t/se-0414-region-based-isolation/68805)), ([revision](https://forums.swift.org/t/returned-for-revision-se-0414-region-based-isolation/69123)), ([second review](https://forums.swift.org/t/se-0414-second-review-region-based-isolation/69740)), ([acceptance](https://forums.swift.org/t/accepted-with-modifications-se-0414-region-based-isolation/70051))
99

1010
## Introduction
1111

proposals/0417-task-executor-preference.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
* Review Manager: [Doug Gregor](https://github.com/DougGregor)
66
* Status: **Accepted**
77
* Implementation: [PR #68793](https://github.com/apple/swift/pull/68793)
8-
* Review: [Acceptance](https://forums.swift.org/t/accepted-se-0417-task-executor-preference/69705), [Review](https://forums.swift.org/t/se-0417-task-executor-preference/68958), [Pitch](https://forums.swift.org/t/pitch-task-executor-preference/68191)
8+
* Review: ([pitch](https://forums.swift.org/t/pitch-task-executor-preference/68191)), ([review](https://forums.swift.org/t/se-0417-task-executor-preference/68958)), ([acceptance](https://forums.swift.org/t/accepted-se-0417-task-executor-preference/69705))
99

1010
## Introduction
1111

proposals/0422-caller-side-default-argument-macro-expression.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
* Review Manager: [Doug Gregor](https://github.com/DougGregor)
66
* Status: **Implemented (Swift 6.0)**
77
* Implementation: [PR #70602](https://github.com/apple/swift/pull/70602) with experimental feature flag `ExpressionMacroDefaultArguments`
8-
* Review: ([Review](https://forums.swift.org/t/se-0422-expression-macro-as-caller-side-default-argument/69730))([Pitch](https://forums.swift.org/t/pitch-expression-macro-as-caller-side-default-argument/69019))([Acceptance](https://forums.swift.org/t/accepted-se-0422-expression-macro-as-caller-side-default-argument/70050))
8+
* Review: ([pitch](https://forums.swift.org/t/pitch-expression-macro-as-caller-side-default-argument/69019)), ([review](https://forums.swift.org/t/se-0422-expression-macro-as-caller-side-default-argument/69730)), ([acceptance](https://forums.swift.org/t/accepted-se-0422-expression-macro-as-caller-side-default-argument/70050))
99

1010
## Introduction
1111

0 commit comments

Comments
 (0)