Skip to content

[Import as member] Core Graphics #2071

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Apr 6, 2016
Merged

Conversation

milseman
Copy link
Member

@milseman milseman commented Apr 5, 2016

What's in this pull request?

Introduces CoreGraphics.apinotes, in which we enable the
import-as-member inference system. Additionally, include some explicit
SwiftNames, for when inference doesn't produce the right result, and
to aid compatibility with the overlays.

Resolved bug number: (SR-)


Before merging this pull request to apple/swift repository:

  • Test pull request on Swift continuous integration.

Triggering Swift CI

The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:

Smoke Testing

Platform Comment
All supported platforms @swift-ci Please smoke test
OS X platform @swift-ci Please smoke test OS X platform
Linux platform @swift-ci Please smoke test Linux platform

Validation Testing

Platform Comment
All supported platforms @swift-ci Please test
OS X platform @swift-ci Please test OS X platform
Linux platform @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@milseman
Copy link
Member Author

milseman commented Apr 5, 2016

@swift-ci please test

@milseman
Copy link
Member Author

milseman commented Apr 5, 2016

ERROR: Couldn't find any revision to build. Verify the repository and branch configuration for this job.
@shahmishal what's going on? Is this related to github's recent downtime?

@milseman
Copy link
Member Author

milseman commented Apr 5, 2016

Alright, now that github is functioning normally again,
@swift-ci please test

@milseman
Copy link
Member Author

milseman commented Apr 5, 2016

@swift-ci please test

@milseman
Copy link
Member Author

milseman commented Apr 6, 2016

@swift-ci please test

@milseman
Copy link
Member Author

milseman commented Apr 6, 2016

Ugh, this is failing due to SDK differences and I can't seem to repro right now. I pushed a fix to this specific issue, and now I'll be preparing my machine to be more like the builder's set up.

@swift-ci please test

milseman added 6 commits April 6, 2016 11:03
Fixes a bug where we weren't performing enough checks on inferred
computed properties that did not have a pair-ed up setter.
Introduces CoreGraphics.apinotes, in which we enable the
import-as-member inference system. Additionally, include some explicit
SwiftNames, for when inference doesn't produce the right result, and
to aid compatibility with the overlays.

Refactors many of the trivial overlays out, shrinking the
CoreGraphics.swift overlay by over half. Updates in-tree test
cases. The names we currently have will be highly in flux for a while,
and are likely to change frequently over the near term.

There are a few remaining known bugs that are worked around by
apinotes entries.
@milseman
Copy link
Member Author

milseman commented Apr 6, 2016

@swift-ci please test

@milseman milseman merged commit b0d54b1 into swiftlang:master Apr 6, 2016
@milseman milseman deleted the coregraphics branch April 7, 2016 20:22
MaxDesiatov pushed a commit that referenced this pull request Apr 19, 2021
[pull] swiftwasm from main
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.

2 participants