pointfreeco / swift-snapshot-testing

📸 Delightful Swift snapshot testing.
https://www.pointfree.co/episodes/ep41-a-tour-of-snapshot-testing
MIT License
3.78k stars 580 forks source link

Update swift-syntax dependency's package url to https://github.com/swiftlang/swift-syntax #878

Closed seankit closed 3 months ago

seankit commented 3 months ago

Describe the bug

It was announced at WWDC that the Swift project would be moving GitHub organizations, from apple to swiftlang. They wrote about it in some detail on this blog post. It looks like swift-syntax recently completed the migration, and as such the url has changed to https://github.com/swiftlang/swift-syntax.

If a Swift Package has both swift-syntax and swift-snapshot-testing as dependencies, this build warning is given:

'swift-snapshot-testing' dependency on 'https://github.com/apple/swift-syntax' conflicts with dependency on 'https://github.com/swiftlang/swift-syntax' which has the same identity 'swift-syntax'. this will be escalated to an error in future versions of SwiftPM.

To Reproduce Demonstration.zip

Expected behavior Swift Package should build without a conflicting identity warning.

Screenshots N/A

Environment

Additional context N/A

stephencelis commented 3 months ago

@seankit This rolled out in https://github.com/pointfreeco/swift-snapshot-testing/releases/tag/1.17.2

seankit commented 3 months ago

whoops I missed that somehow, thank you!