Skip to content
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

Protocol 'ExpectedCount' cannot be nested inside another declaration #118

Closed
andrzejplatek opened this issue Aug 20, 2024 · 4 comments · Fixed by #119
Closed

Protocol 'ExpectedCount' cannot be nested inside another declaration #118

andrzejplatek opened this issue Aug 20, 2024 · 4 comments · Fixed by #119

Comments

@andrzejplatek
Copy link

When updating the package to version 1.2.4 we were unable to build a package. I could reproduce the issue by adding a package to new empty project.

Swift 5.9
Xcode version: 15.2

Zrzut ekranu 2024-08-20 o 10 09 36

@MartinMetselaar
Copy link

Nested protocols have been introduced since Swift 5.10:
https://github.com/swiftlang/swift-evolution/blob/main/proposals/0404-nested-protocols.md

So you can start using the latest swift version to resolve the issue. I don't know what the guidelines are for this project to support older swift versions but I was encountering the same issue.

@andrzejplatek
Copy link
Author

andrzejplatek commented Aug 20, 2024

Thanks, i guess Swift.package should be updated to swift-tools-version: 5.10 in that case

@mbrandonw
Copy link
Member

Hi @andrzejplatek, thanks for the report! That was a mistake on our part. We seem to have lost Swift 5.9 CI and so it did not let us know of the problem. Hopefully this #119 will fix things and we can make a new release soon.

@GoodyGoodies
Copy link

As a temporary decision try to put
.package(url: "https://github.com/pointfreeco/xctest-dynamic-overlay", exact: "1.2.3"),
into your dependencies to let the deps manager do the rest

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 a pull request may close this issue.

4 participants