Skip to content

Conversation

@elsh
Copy link
Contributor

@elsh elsh commented Mar 31, 2023

Support Target Grouping

  • Allow opting out of package boundary
  • Default set to package boundary

Resolves rdar://107454728

@elsh
Copy link
Contributor Author

elsh commented Mar 31, 2023

@swift-ci smoke test

@elsh elsh changed the title Support Target Grouping [5.9] Support Target Grouping Mar 31, 2023
@elsh elsh added the swift 5.9 This PR targets the 5.9 branch label Mar 31, 2023
@elsh elsh self-assigned this Mar 31, 2023
- Allow opting out of package boundary
- Default set to package boundary

Resolves rdar://107454728
@elsh
Copy link
Contributor Author

elsh commented Mar 31, 2023

@swift-ci smoke test

@elsh
Copy link
Contributor Author

elsh commented Apr 3, 2023

@swift-ci smoke test windows

@elsh elsh requested a review from nkcsgexi April 3, 2023 23:24
@elsh
Copy link
Contributor Author

elsh commented Apr 3, 2023

cherrypicked from main #6356

@elsh elsh merged commit 82b08e1 into release/5.9 Apr 4, 2023
@available(_PackageDescription, introduced: 999.0)
static func macro(
name: String,
group: TargetGroup,
Copy link
Member

Choose a reason for hiding this comment

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

This breaks existing macro packages, and deviates from the API under review in SE-0394. Can we please make this optional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

swift 5.9 This PR targets the 5.9 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants