forked from swiftlang/swift-evolution
-
Notifications
You must be signed in to change notification settings - Fork 0
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
[pull] main from apple:main #1
Open
pull
wants to merge
468
commits into
ScorpiusDraconis83:main
Choose a base branch
from
swiftlang:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: John McCall <[email protected]>
The current Swift Package Manager manifest API for specifying Swift language version(s) applies to an entire package which is limiting when adopting new language versions that have implications for source compatibility. Swift-evolution thread: https://forums.swift.org/t/pitch-swiftpm-swift-language-version-per-target/71067
Co-authored-by: John McCall <[email protected]>
* Proposal to introduce borrowing switch for noncopyable types * Revised design: default to borrowing, and have `let` following the prevailing ownership of the borrow * Kick off review --------- Co-authored-by: Ben Cohen <[email protected]>
* Revise SE-0423 based on review feedback. * Kick off second review --------- Co-authored-by: Ben Cohen <[email protected]>
* Pitch Mutex Update nnnn-mutex.md Updates to mutex fix indents more indent fixes some more moot changes Some updates from feedback Mark a section as TODO Remove lock, unlock, tryLock Rewrite motivation and define differences between actors Apply suggestions from code review Co-authored-by: Konrad `ktoso` Malawski <[email protected]> Remove reference to old free lock and unlock Remove `@Sendable` on tryWithLockUnchecked * Change tryWithLock to withLockIfAvailable * Discuss disconnected as the future direction * Update nnnn-mutex.md * Discuss the transferring inout Add link to transferring proposal Fix spaces Fix spaces Add missing optional Add missing optional * Rename generic argument to State * Fix Sendable extension and move it up * Add toolchains * Add note about toolchains not having transferring inout * Update proposals/nnnn-mutex.md * Update and rename nnnn-mutex.md to 0433-mutex.md --------- Co-authored-by: Stephen Canon <[email protected]>
* Update 0433-mutex.md Change position of bracketed phrase * Update 0433-mutex.md leans / lends * Update 0433-mutex.md added 'which' for clarity * Update 0433-mutex.md actoe / actor * Update proposals/0433-mutex.md * Update proposals/0433-mutex.md --------- Co-authored-by: Stephen Canon <[email protected]>
…usability Add a proposal to improve usability of global-actor-isolated types.
Link SE-0434 to its review thread
- Make implementation an explicit link so it appears on SE Dashboard - Use https for review manager link
For these proposals, also: - Normalize discussion naming according to template instructions (e.g. 'first pitch', 'acceptance', 'returned' - Normalize use of 'Review' heading label
Return SE-0430 for revision
Small typo around code voice usage.
Update 0418-inferring-sendable-for-methods.md
[BitwiseCopyable] Post-review changes.
…-terminology Proposal to formalize 'language mode' terminology
Link SE-0441 to its review thread
* Update proposals that are implemented in Swift 6.0 * More updates and remove implementation fields. * Remove experimental implementation details from SE-0425.
* [SE-0440] Second revision * Add Revision history section * Update 0440-debug-description-macro.md * Update 0440-debug-description-macro.md --------- Co-authored-by: Stephen Canon <[email protected]>
[SE-0438] Metatype Keypaths proposal is accepted
* Initial WIP proposal document. * Minor updates. * Update proposal document with suggstions. * Update implementation link. * Add GitHub user links. * Update to provide addTask(...) closure bodies and provide clarity in the first sentence of the Design Details section. * Update examples.
* Updated proposal - synchronous deinit is nonisolated by default - async deinit - attribute to control task-local usage * Minimized "Interaction with ObjC runtime" section * Removed C++ code from "Runtime" section * Removed assertion from future directions Already implemented in swiftlang/swift@108f780 * Reviewed proposal text * Added paragraph about task-local values and future async deinit optimizations * Removed async deinit from the proposal * Make task-locals undefined for isolated deinit * Removed questionable paragraph from the Motivation section * [SE-0371] Address some concerns about nondeterminism in motivation * Removed few more references to async deinit * Applied review comments --------- Co-authored-by: Konrad `ktoso` Malawski <[email protected]> Co-authored-by: Frederick Kellison-Linn <[email protected]>
Minor spelling mistake.
Fixed typo compatibility
…ve-review Update SE-0371 status to reflect active second review
…discussion Add link to second review thread for SE-0440
Update status field. Add review links.
Co-authored-by: Doug Gregor <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )