Skip to content

Conversation

cwakamo
Copy link
Contributor

@cwakamo cwakamo commented Jan 29, 2018

This pull request removes the PlaygroundQuickLook enum, the CustomPlaygroundQuickLookable protocol, and the runtime pieces which supported PlaygroundQuickLook.init(reflecting:).

This is the implementation of an upcoming swift-evolution proposal which aims to remove these types from the standard library and replace them with a new CustomPlaygroundDisplayConvertible protocol in the PlaygroundSupport library.

@cwakamo
Copy link
Contributor Author

cwakamo commented Jan 29, 2018

Please test with the following PRs:

swiftlang/swift-corelibs-foundation#1415
apple/swift-xcode-playground-support#20

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 80ff99039960694111d5eab2eacacc67f0a33ea2

@cwakamo
Copy link
Contributor Author

cwakamo commented Jan 30, 2018

@swift-ci Please test OS X platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 80ff99039960694111d5eab2eacacc67f0a33ea2

@cwakamo
Copy link
Contributor Author

cwakamo commented Jan 30, 2018

Please test with the following PRs:

swiftlang/swift-corelibs-foundation#1415
apple/swift-xcode-playground-support#20

@swift-ci Please test OS X platform

@cwakamo
Copy link
Contributor Author

cwakamo commented Jan 30, 2018

Please test with the following PRs:

swiftlang/swift-corelibs-foundation#1415
apple/swift-xcode-playground-support#20

@swift-ci Please test source compatibility

…overlay.

The CustomPlaygroundQuickLookable conformances have been deleted, so this is no longer required.
`import CoreGraphics` is required for this test on non-macOS Apple platforms. (On macOS, `import Foundation` is sufficient.)
This previously was not an issue, as this test used to `import UIKit` on non-macOS Apple platforms.
Since CoreGraphics is available everywhere this test runs, and this test does test CoreGraphics types, I just added an import of CoreGraphics.
@cwakamo cwakamo force-pushed the remove-PlaygroundQuickLook-and-friends branch from dc70faf to 68cca77 Compare February 6, 2018 20:04
@cwakamo
Copy link
Contributor Author

cwakamo commented Feb 6, 2018

Please test with the following PRs:

swiftlang/swift-corelibs-foundation#1415
apple/swift-xcode-playground-support#20

@swift-ci Please test

@swift-ci
Copy link
Contributor

swift-ci commented Feb 6, 2018

Build failed
Swift Test Linux Platform
Git Sha - 80ff99039960694111d5eab2eacacc67f0a33ea2

@swift-ci
Copy link
Contributor

swift-ci commented Feb 6, 2018

Build failed
Swift Test OS X Platform
Git Sha - dc70faf74264a64b35ac9b15786331fa7dbbf495

@cwakamo
Copy link
Contributor Author

cwakamo commented Feb 6, 2018

Please test with the following PRs:

swiftlang/swift-corelibs-foundation#1415
apple/swift-xcode-playground-support#20

@swift-ci Please test source compatibility

@CodaFi CodaFi added the swift evolution pending discussion Flag → feature: A feature that has a Swift evolution proposal currently in review label Nov 18, 2019
@natecook1000
Copy link
Member

@cwakamo Was this PR superseded by another? The proposal says it was accepted and implemented.

@cwakamo
Copy link
Contributor Author

cwakamo commented Feb 2, 2020

Closing this PR as we did not end up going in this direction.

@cwakamo cwakamo closed this Feb 2, 2020
@AnthonyLatsis
Copy link
Collaborator

Proposal: SE-0198

@AnthonyLatsis AnthonyLatsis removed the swift evolution pending discussion Flag → feature: A feature that has a Swift evolution proposal currently in review label Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants