Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Conversation

@crazytonyli
Copy link
Contributor

Description

Part of re-implementing Objective-C in Swift.

  • Please check here if your pull request includes additional test coverage.
  • I have considered updating the version in the .podspec file.

@crazytonyli crazytonyli requested a review from mokagio December 8, 2022 08:03
Copy link
Contributor

@mokagio mokagio left a comment

Choose a reason for hiding this comment

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

Neat little PR 👍

All those ? make me worry, but at least the compiler will make consumers deal with them...

When do you think it would be a good time to address all that nullability? Or, better question, is that something we should be worrying about at all (in the context of limited resources)

@@ -0,0 +1,11 @@
import Foundation

@objcMembers public class RemoteMenu: NSObject {
Copy link
Contributor

Choose a reason for hiding this comment

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

TIL about @objcMembers, to make all members of a Swift type available to Objective-C. Neat.

Base automatically changed from update-nsobject-expecations to trunk December 8, 2022 20:27
@crazytonyli crazytonyli merged commit fc7d2ab into trunk Dec 8, 2022
@crazytonyli crazytonyli deleted the swift-translation-remote-menu branch December 8, 2022 20:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants