Skip to content

Conversation

@TomerHacohen
Copy link
Contributor

@TomerHacohen TomerHacohen commented Jun 25, 2024

Fixes the warning in the new Swift version solved here:
swiftlang/swift#36068

Which is:

Extension declares a conformance of imported type 'X' to imported protocol 'Y'; this will not behave correctly if the owners of 'CoreFoundation' introduce this conformance in the future

without adding the @retroactive, which does not compile in Xcode 15.

This way, extensions that declares a conformance to a type where both are from another module will compile without warning in Xcode 16, as the conformed type is declared to be explicitly from the imported module, which suppresses the warning.

Copy link
Member

@younata younata left a comment

Choose a reason for hiding this comment

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

Thanks!

@younata younata merged commit 6ba9e68 into Quick:main Jun 25, 2024
@TomerHacohen TomerHacohen deleted the feature/fix-retroactive-conformance-error-in-xcode16 branch June 26, 2024 06:13
cgrindel-self-hosted-renovate bot referenced this pull request in cgrindel/rules_swift_package_manager Aug 29, 2024
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [Quick/Nimble](https://togithub.com/Quick/Nimble) | minor | `from:
"13.3.0"` -> `from: "13.4.0"` |

---

### Release Notes

<details>
<summary>Quick/Nimble (Quick/Nimble)</summary>

### [`v13.4.0`](https://togithub.com/Quick/Nimble/releases/tag/v13.4.0)

[Compare
Source](https://togithub.com/Quick/Nimble/compare/v13.3.0...v13.4.0)

#### What's Changed

- Bump rexml from 3.2.6 to 3.2.8 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/Quick/Nimble/pull/1143](https://togithub.com/Quick/Nimble/pull/1143)
- BeLogical: fix retroactive conformance error. by
[@&#8203;TomerHacohen](https://togithub.com/TomerHacohen) in
[https://github.com/Quick/Nimble/pull/1149](https://togithub.com/Quick/Nimble/pull/1149)
- Add a matcher to negate a passed in matcher by
[@&#8203;younata](https://togithub.com/younata) in
[https://github.com/Quick/Nimble/pull/1151](https://togithub.com/Quick/Nimble/pull/1151)

#### New Contributors

- [@&#8203;TomerHacohen](https://togithub.com/TomerHacohen) made their
first contribution in
[https://github.com/Quick/Nimble/pull/1149](https://togithub.com/Quick/Nimble/pull/1149)

**Full Changelog**:
Quick/Nimble@v13.3.0...v13.4.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config help](https://togithub.com/renovatebot/renovate/discussions) if
that's undesired.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC41NS4wIiwidXBkYXRlZEluVmVyIjoiMzguNTUuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

Co-authored-by: cgrindel-self-hosted-renovate[bot] <139595543+cgrindel-self-hosted-renovate[bot]@users.noreply.github.com>
noamfreeman pushed a commit to noamfreeman/Nimble that referenced this pull request Oct 14, 2024
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