Skip to content

Fix noCacheKeyConformance error on make-linux-sdk #97

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 1 commit into from
Apr 4, 2024

Conversation

kateinoigakukun
Copy link
Member

Follow-up to 3e076d1

SystemString used inside FilePath is not a CacheKey, so the encoder complains about it, but we can consider FilePath as a leaf type and no need to encode it further.

Follow-up to 3e076d1

SystemString used inside FilePath is not a CacheKey, so the encoder
complains about it, but we can consider FilePath as a leaf type and no need
to encode it further.
Copy link
Contributor

@MaxDesiatov MaxDesiatov left a comment

Choose a reason for hiding this comment

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

Thanks!

@MaxDesiatov
Copy link
Contributor

@swift-ci test

@MaxDesiatov MaxDesiatov merged commit 4c2e16f into swiftlang:main Apr 4, 2024
@MaxDesiatov MaxDesiatov added the needs tests The change regresses test coverage label Aug 28, 2024
MaxDesiatov added a commit that referenced this pull request Aug 28, 2024
Follow-up to #97 with an added test case.
MaxDesiatov added a commit to swiftlang/swift-package-manager that referenced this pull request Aug 28, 2024
This vendors fixes to `HashEncoder` from `swift-sdk-generator` introduced in swiftlang/swift-sdk-generator#97 and swiftlang/swift-sdk-generator#134.
MaxDesiatov added a commit that referenced this pull request Aug 28, 2024
Follow-up to #97 with an added test case.
MaxDesiatov added a commit to swiftlang/swift-package-manager that referenced this pull request Aug 28, 2024
This vendors fixes to `HashEncoder` from `swift-sdk-generator` introduced in swiftlang/swift-sdk-generator#97 and swiftlang/swift-sdk-generator#134.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs tests The change regresses test coverage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants