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

Add support for package visibility modifier #1472

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

gjcairo
Copy link
Contributor

@gjcairo gjcairo commented Sep 26, 2023

This PR adds support for the new package visibility modifier, introduced with Swift 5.9.
It also adds support for Swift 5.9, by updating the Package.swift's swift-tools-versiont o 5.9 (required to use package) and moving the existing Package.swift to Package@swift-5.8.swift.

@gjcairo gjcairo marked this pull request as ready for review September 26, 2023 13:47
Copy link
Member

@FranzBusch FranzBusch left a comment

Choose a reason for hiding this comment

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

We also need to extend the Plugin.swift to support this.

Sources/protoc-gen-swift/GeneratorOptions.swift Outdated Show resolved Hide resolved
@gjcairo gjcairo force-pushed the package-visibility branch 2 times, most recently from 68c6a02 to e2c79d5 Compare September 26, 2023 14:10
@gjcairo gjcairo requested a review from FranzBusch September 26, 2023 14:11
Package.swift Outdated Show resolved Hide resolved
Package@swift-5.7.swift Outdated Show resolved Hide resolved
@gjcairo gjcairo requested a review from thomasvl September 28, 2023 07:25
Copy link
Member

@FranzBusch FranzBusch left a comment

Choose a reason for hiding this comment

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

LGTM!

@thomasvl thomasvl merged commit 99808ee into apple:main Sep 28, 2023
9 checks passed
thomasvl added a commit to thomasvl/swift-protobuf that referenced this pull request Jan 2, 2024
Follow up to apple#1472, looks like we didn't realize this gate should also be updated.
thomasvl added a commit that referenced this pull request Jan 3, 2024
Follow up to #1472, looks like we didn't realize this gate should also be updated.
thomasvl added a commit to thomasvl/swift-protobuf that referenced this pull request Jan 3, 2024
Follow up to apple#1472, looks like we didn't realize this gate should also be updated.
thomasvl added a commit that referenced this pull request Jan 3, 2024
Follow up to #1472, looks like we didn't realize this gate should also be updated.
@gjcairo gjcairo deleted the package-visibility branch March 27, 2024 16:02
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 this pull request may close these issues.

3 participants