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

[6.0][PackageDescription] SE-0441: Rename swiftLanguageVersion to `swift… #7813

Merged
merged 2 commits into from
Jul 24, 2024

Conversation

xedin
Copy link
Contributor

@xedin xedin commented Jul 23, 2024

…LanguageMode`

  • Explanation:

    Implementation of SE-0441.

    • Deprecates swiftLanguageVersion API to maintain source compatibility with existing manifests;
    • Introduces new PackageDescription API - swiftLanguageMode (available from PackageDescription 6 on);
    • Renames multiple internal APIs to use mode instead of version when appropriate;
    • Build setting is still -swift-version but would be changed to use mode when it becomes available.
  • Main Branch PR: Adopt language mode naming in PackageDescription API #7620

  • Risk: Low (Diagnoses invalid code instead of crashing and allows more uses of init accessors in accordance with the proposal).

  • Reviewed By: @MaxDesiatov @xedin @bnbarham

  • Testing: Existing test-cases were modified and new tests were added.

(cherry picked from commit 844156b1e5f2d219541f7487f0d6e3d33890f139)

…LanguageMode`

- Deprecates `swiftLanguageVersion` API to maintain source compatibility with existing manifests;
- Introduces new PackageDescription API - `swiftLanguageMode` (available from PackageDescription 6 on);
- Renames multiple internal APIs to use `mode` instead of `version` when appropriate;
- Build setting is still `-swift-version` but would be changed to use `mode` when it becomes available.
@xedin xedin requested a review from bnbarham July 23, 2024 21:13
@xedin
Copy link
Contributor Author

xedin commented Jul 23, 2024

@swift-ci please test

…sions:` initializer

Allow `swiftLanguageVersions:` if it's explicitly specified to
maintain source compatibility with existing manifests.
@xedin
Copy link
Contributor Author

xedin commented Jul 23, 2024

@swift-ci please test

@xedin
Copy link
Contributor Author

xedin commented Jul 23, 2024

@swift-ci please test Windows platform

@bnbarham bnbarham marked this pull request as ready for review July 24, 2024 04:54
@bnbarham bnbarham requested a review from a team as a code owner July 24, 2024 04:54
@bnbarham bnbarham merged commit c6dedd5 into swiftlang:release/6.0 Jul 24, 2024
5 checks passed
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