Skip to content
This repository was archived by the owner on Mar 26, 2023. It is now read-only.

[64] SE-0162 under review #207

Merged
merged 2 commits into from
Apr 5, 2017
Merged

[64] SE-0162 under review #207

merged 2 commits into from
Apr 5, 2017

Conversation

BasThomas
Copy link
Contributor

Also put SE-0155 on top :)

Also put SE-0155 on top :)
@hal-codebot
Copy link

hal-codebot commented Apr 4, 2017

Proselint found issues

_drafts/2017-04-06-issue-64.md

Line Message Severity
8 More than 30 ppm of exclamations. Keep them under control. warning
31 Annotation left in text. warning
84 Substitute 'damn' every time you're inclined to write 'very;' your editor will delete it and the writing will be just as it should be. Found once elsewhere. warning
84 No hyphen is necessary in phrasal adjectives with an adverb ending in -ly, unless the -ly adverb is part of a longer phrase warning
88 'hopefully' is a bit of a skunked term — impossible to use without issue. Find some other way to say it. warning
107 No hyphen is necessary in phrasal adjectives with an adverb ending in -ly, unless the -ly adverb is part of a longer phrase warning

Spell Checker found issues

_drafts/2017-04-06-issue-64.md

Line Typo
19 Episode 5: [Objective-C Interoperabilit
25 JP Simard [noticed](https://twit
25 JP Simard [noticed](https://twitter.com
25 ter/tools/SourceKit) in Xcode 8.3, and predicted we'll see refa
25 ng support for Swift in Xcode 9. He also [pointed out](https:
27 Speaking of JP Simard, he released [SourceKi
27 Speaking of JP Simard, he released [SourceKitten 0.
27 ng of JP Simard, he released [SourceKitten 0.17.1](https://github.com/jp
27 rd, he released [SourceKitten 0.17.1](https://github.com/jpsim/Sou
27 0.17.1), which includes Swift 3.1 support.
35 [SE-0159](https://github.com/apple/swi
39 > [SE-0159](https://github.com/apple/swi
39 the main user-facing part of [SE-0025](https://github.com/apple/swi
39 was sufficient evidence that more-restrictive-than-fileprivate access control is in use with
39 functionality introduced by [SE-0025](https://github.com/apple/swi
41 ny of the concerns raised in [SE-0159](https://github.com/apple/swi
46 code bases moving from Swift 3 to Swift 4, compromising the
46 moving from Swift 3 to Swift 4, compromising the source stab
46 ility goals set out for Swift 4.
48 into a new proposal for Swift 4.
52 [SE-0155](https://github.com/apple/swi
52 epresentation.md): Normalize Enum Case Representation by Danie
52 ase Representation* by Daniel Duan and Joe Groff is [under revie
54 > In Swift 3, associated values of an enum
54 ft 3, associated values of an enum case are represented by a tup
56 > Enums, therefore, can be made more
58 -evolution thread: [Normalize Enum Case Representation (rev. 2)]
58 num Case Representation (rev. 2)](https://lists.swift.org/pip
60 [SE-0161](https://github.com/apple/swi
60 ls/0161-key-paths.md): *Smart KeyPaths: Better Key-Value Coding for
60 wift* by David Smith, Michael LeHew, and Joe Groff is [under revi
62 ey Path_ types that represent uninvoked references to properties that
73 and in the future additional behaviors.
75 > More Expressive KeyPaths
78 [SE-0162](https://github.com/apple/swi
78 ger Custom Target Layouts* by Ankit Aggarwal is [under review](ht
78 stom Target Layouts* by Ankit Aggarwal is [under review](https://lis
88 ned above in the rejection of SE-0159, Doug Gregor has [opened a ne
88 he rejection of SE-0159, Doug Gregor has [opened a new discussion]
90 > In rejecting [SE-0159](https://github.com/apple/swi
90 lines for review in the Swift 4 time-frame (i.e., very soon).
101 this was going to be true of [SE-0025](https://github.com/apple/swi

Generated by 🚫 Danger


> In Swift 3, associated values of an enum case are represented by a tuple. This implementation causes inconsistencies in case declaration, construction and pattern matching in several places.
>
> Enums, therefore, can be made more "regular" when we replace tuple as the representation of associated case values. This proposal aims to define the effect of doings so on various parts of the language.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"doing so"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch! Opened a PR on swift-evolution as well.

@jessesquires jessesquires merged commit 5c90062 into master Apr 5, 2017
@jessesquires jessesquires deleted the BasThomas-patch-1 branch April 5, 2017 04:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

4 participants