Skip to content

Commit fa26c20

Browse files
Normalize review metadata
- Make bare discussion links into explicit links - Add missing pitch, review, and acceptance links as applicable - Change field order to match proposal template - Add explicit implementation links to SE-0391
1 parent 43b894b commit fa26c20

File tree

4 files changed

+16
-20
lines changed

4 files changed

+16
-20
lines changed

proposals/0099-conditionclauses.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
* Authors: [Erica Sadun](https://github.com/erica), [Chris Lattner](https://github.com/lattner)
55
* Review Manager: [Joe Groff](https://github.com/jckarter)
66
* Status: **Implemented (Swift 3.0)**
7-
* Decision Notes: [Rationale](#rationale)
87
* Previous Revision: [1](https://github.com/apple/swift-evolution/blob/83053c5f5395987caf2ecb3830a5cd8dc6213237/proposals/0099-conditionclauses.md)
8+
* Review: ([pitch](https://forums.swift.org/t/pitch-making-where-and-interchangeable-in-guard-conditions/2702)), ([review](https://forums.swift.org/t/review-se-0099-restructuring-condition-clauses/2808)), ([acceptance](https://forums.swift.org/t/accepted-with-revision-se-0099-restructuring-condition-clauses/2921))
99

1010
## Introduction
1111

proposals/0363-unicode-for-string-processing.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
* Authors: [Nate Cook](https://github.com/natecook1000), [Alejandro Alonso](https://github.com/Azoy)
55
* Review Manager: [Ben Cohen](https://github.com/airspeedswift)
66
* Status: **Implemented (Swift 5.7)**
7-
* Review: https://forums.swift.org/t/se-0363-unicode-for-string-processing/58520
8-
* Implementation: [apple/swift-experimental-string-processing][repo]
7+
* Implementation: [apple/swift-experimental-string-processing][repo]
8+
* Review: ([pitch](https://forums.swift.org/t/pitch-unicode-for-string-processing/56907)), ([review](https://forums.swift.org/t/se-0363-unicode-for-string-processing/58520)), ([acceptance](https://forums.swift.org/t/accepted-se-0363-unicode-for-string-processing/59998))
99

1010
### Version History
1111

proposals/0378-package-registry-auth.md

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,7 @@
55
* Review Manager: [Tom Doron](https://github.com/tomerd)
66
* Status: **Implemented (Swift 5.8)**
77
* Implementation: [apple/swift-package-manager#5838](https://github.com/apple/swift-package-manager/pull/5838)
8-
* Review:
9-
* Pitch: https://forums.swift.org/t/pitch-package-registry-authentication/61047
10-
* Review: https://forums.swift.org/t/se-0378-swift-package-registry-authentication/61436
8+
* Review: ([pitch](https://forums.swift.org/t/pitch-package-registry-authentication/61047)), ([review](https://forums.swift.org/t/se-0378-swift-package-registry-authentication/61436)), ([acceptance](https://forums.swift.org/t/accepted-with-modifications-se-0378-swift-package-registry-authentication/62556))
119

1210
## Introduction
1311

proposals/0391-package-registry-publish.md

Lines changed: 12 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -5,20 +5,18 @@
55
* Review Manager: [Tom Doron](https://github.com/tomerd)
66
* Status: **Implemented (Swift 5.9)**
77
* Implementation:
8-
* https://github.com/apple/swift-package-manager/pull/6101
9-
* https://github.com/apple/swift-package-manager/pull/6146
10-
* https://github.com/apple/swift-package-manager/pull/6159
11-
* https://github.com/apple/swift-package-manager/pull/6169
12-
* https://github.com/apple/swift-package-manager/pull/6188
13-
* https://github.com/apple/swift-package-manager/pull/6189
14-
* https://github.com/apple/swift-package-manager/pull/6215
15-
* https://github.com/apple/swift-package-manager/pull/6217
16-
* https://github.com/apple/swift-package-manager/pull/6220
17-
* https://github.com/apple/swift-package-manager/pull/6229
18-
* https://github.com/apple/swift-package-manager/pull/6237
19-
* Review:
20-
* pitch: https://forums.swift.org/t/pitch-package-registry-publish/62828
21-
* review: https://forums.swift.org/t/se-0391-package-registry-publish/63405
8+
* [apple/swift-package-manager#6101](https://github.com/apple/swift-package-manager/pull/6101)
9+
* [apple/swift-package-manager#6146](https://github.com/apple/swift-package-manager/pull/6146)
10+
* [apple/swift-package-manager#6159](https://github.com/apple/swift-package-manager/pull/6159)
11+
* [apple/swift-package-manager#6169](https://github.com/apple/swift-package-manager/pull/6169)
12+
* [apple/swift-package-manager#6188](https://github.com/apple/swift-package-manager/pull/6188)
13+
* [apple/swift-package-manager#6189](https://github.com/apple/swift-package-manager/pull/6189)
14+
* [apple/swift-package-manager#6215](https://github.com/apple/swift-package-manager/pull/6215)
15+
* [apple/swift-package-manager#6217](https://github.com/apple/swift-package-manager/pull/6217)
16+
* [apple/swift-package-manager#6220](https://github.com/apple/swift-package-manager/pull/6220)
17+
* [apple/swift-package-manager#6229](https://github.com/apple/swift-package-manager/pull/6229)
18+
* [apple/swift-package-manager#6237](https://github.com/apple/swift-package-manager/pull/6237)
19+
* Review: ([pitch](https://forums.swift.org/t/pitch-package-registry-publish/62828)), ([review](https://forums.swift.org/t/se-0391-package-registry-publish/63405)), ([acceptance](https://forums.swift.org/t/accepted-se-0391-swift-package-registry-authentication/64088))
2220

2321
## Introduction
2422

0 commit comments

Comments
 (0)