Skip to content

Conversation

@xedin
Copy link
Contributor

@xedin xedin commented Mar 24, 2025

No description provided.

@xedin
Copy link
Contributor Author

xedin commented Mar 24, 2025

@swift-ci please test

[#7741]: https://github.com/swiftlang/swift-package-manager/pull/7741
[#7813]: https://github.com/swiftlang/swift-package-manager/pull/7813
[#8301]: https://github.com/swiftlang/swift-package-manager/pull/8301
[#8314]: https://github.com/swiftlang/swift-package-manager/pull/8314
Copy link
Contributor

Choose a reason for hiding this comment

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

suggestion: This link is not referenced in the update. Can we either (1) remove it, or (2) use the reference?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's just a fix for a previous entry about strictMemorySafety.

Copy link
Contributor

Choose a reason for hiding this comment

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

(ie. (2) is already true)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, sorry I wasn't clear - 8314 is used by strictMemorySafety entry, it just wasn't declared here.

@xedin
Copy link
Contributor Author

xedin commented Mar 24, 2025

@swift-ci please test macOS platform

[#7741]: https://github.com/swiftlang/swift-package-manager/pull/7741
[#7813]: https://github.com/swiftlang/swift-package-manager/pull/7813
[#8301]: https://github.com/swiftlang/swift-package-manager/pull/8301
[#8314]: https://github.com/swiftlang/swift-package-manager/pull/8314
Copy link
Contributor

Choose a reason for hiding this comment

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

(ie. (2) is already true)

@xedin xedin merged commit 16b57de into swiftlang:main Mar 24, 2025
5 checks passed
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.

3 participants