Skip to content

Conversation

@grynspan
Copy link
Contributor

@grynspan grynspan commented Jul 2, 2025

The PR (#869) that just added the AppKit overlay for NSImage was a bit stale. This PR corrects a couple of bugs that snuck in due to its age and the resulting bit rot.

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

The PR (#869) that just added the AppKit overlay for `NSImage` was a bit stale.
This PR corrects a couple of bugs that snuck in due to its age and the resulting
bit rot.
@grynspan grynspan added this to the Swift 6.x (main) milestone Jul 2, 2025
@grynspan grynspan self-assigned this Jul 2, 2025
@grynspan grynspan added the bug 🪲 Something isn't working label Jul 2, 2025
@grynspan grynspan requested a review from suzannaratcliff as a code owner July 2, 2025 19:15
@grynspan grynspan added attachments/activities 🖇️ Work related to attachments and/or activities cross-import-overlays 🍰 Cross-import overlays (Foundation, Core Graphics, etc.) labels Jul 2, 2025
@grynspan
Copy link
Contributor Author

grynspan commented Jul 2, 2025

@swift-ci test

#if SWT_TARGET_OS_APPLE && canImport(AppKit)
public import AppKit
@_spi(ForSwiftTestingOnly) @_spi(Experimental) public import _Testing_CoreGraphics
@_spi(Experimental) public import _Testing_CoreGraphics
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess there were not any package declarations needed currently?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right.

@grynspan grynspan merged commit fab283a into main Jul 2, 2025
3 checks passed
@grynspan grynspan deleted the jgrynspan/nsimage-overlay-fixes branch July 2, 2025 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

attachments/activities 🖇️ Work related to attachments and/or activities bug 🪲 Something isn't working cross-import-overlays 🍰 Cross-import overlays (Foundation, Core Graphics, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants