Skip to content

[6.0] NFC: Make 'FileRuleDescription.Rule' conform to Sendable #7781

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
Jul 17, 2024

Conversation

MaxDesiatov
Copy link
Contributor

Explanation: TargetSourcesBuilder.swift: Stored property 'rule' of 'Sendable'-conforming struct 'FileRuleDescription' has non-sendable type 'FileRuleDescription.Rule'; this is an error in the Swift 6 language mode.
Scope: Non-functional change.
Risk: Very low as it's an NFC change removing a compiler warning.
Testing: Manually verified that the warning is resolved.
Issue: rdar://131439553
Reviewer: @bnbarham

**Explanation**: `TargetSourcesBuilder.swift`: Stored property 'rule' of 'Sendable'-conforming struct 'FileRuleDescription' has non-sendable type 'FileRuleDescription.Rule'; this is an error in the Swift 6 language mode.
**Scope**: Non-functional change.
**Risk**: Very low as it's an NFC change removing a compiler warning.
**Testing**: Manually verified that the warning is resolved.
**Issue**: rdar://131439553
**Reviewer**: @bnbarham
@MaxDesiatov MaxDesiatov added concurrency no functional change No user-visible functional changes included labels Jul 16, 2024
@MaxDesiatov MaxDesiatov requested a review from a team as a code owner July 16, 2024 18:26
@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@MaxDesiatov MaxDesiatov added the swift 6.0 Related to Swift 6.0 release branch label Jul 17, 2024
@MaxDesiatov MaxDesiatov changed the title [NFC] Stored property has non-sendable type 'FileRuleDescription.Rule' [6.0] NFC: Make 'FileRuleDescription.Rule' conform to Sendable Jul 17, 2024
@MaxDesiatov MaxDesiatov merged commit 244703f into release/6.0 Jul 17, 2024
5 checks passed
@MaxDesiatov MaxDesiatov deleted the maxd/6.0-sendable-rule branch July 17, 2024 11:23
Hesham-atta

This comment was marked as off-topic.

Hesham-atta

This comment was marked as duplicate.

@swiftlang swiftlang deleted a comment from Hesham-atta Jul 17, 2024
Hesham-atta

This comment was marked as duplicate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
concurrency no functional change No user-visible functional changes included swift 6.0 Related to Swift 6.0 release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants