Skip to content

Conversation

@dfed
Copy link
Owner

@dfed dfed commented Nov 28, 2023

As title. Swift 5.9 introduces new API that allow us to remove our force unwraps!

@dfed dfed requested review from bachand and omv-libs November 28, 2023 23:40
@codecov
Copy link

codecov bot commented Nov 28, 2023

Codecov Report

Merging #22 (1d15a9f) into main (9f2311f) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##              main       #22   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            9         9           
  Lines          936       921   -15     
=========================================
- Hits           936       921   -15     
Files Coverage Δ
Sources/AsyncQueue/ActorQueue.swift 100.00% <100.00%> (ø)
Sources/AsyncQueue/FIFOQueue.swift 100.00% <100.00%> (ø)
Sources/AsyncQueue/MainActorQueue.swift 100.00% <100.00%> (ø)

Copy link
Collaborator

@bachand bachand left a comment

Choose a reason for hiding this comment

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

Nice!

s.authors = 'Dan Federman'
s.source = { :git => 'https://github.com/dfed/swift-async-queue.git', :tag => s.version }
s.swift_version = '5.8'
s.swift_version = '5.9'
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

@@ -1,4 +1,4 @@
// swift-tools-version: 5.8
// swift-tools-version: 5.9
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

// Continuation will be captured during stream creation, so it is safe to force unwrap here.
// If this force-unwrap fails, something is fundamentally broken in the Swift runtime.
taskStreamContinuation = capturedTaskStreamContinuation!
let (taskStream, taskStreamContinuation) = AsyncStream<ActorTask>.makeStream()
Copy link
Collaborator

Choose a reason for hiding this comment

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

🎉

Copy link
Owner Author

Choose a reason for hiding this comment

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

So. Much. Better!

typhoeus (~> 1.0)
cocoapods-deintegrate (1.0.5)
cocoapods-downloader (1.6.3)
cocoapods-downloader (2.1)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a major version update. I am commenting so that this stands out.

Copy link
Owner Author

Choose a reason for hiding this comment

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

This is due to the cocoapods version update. Doesn't affect anyone downstream.

run: sudo xcode-select --switch /Applications/Xcode_14.3.1.app/Contents/Developer
- name: Build and Test Framework
run: xcrun swift test -c release -Xswiftc -enable-testing
run: bundle exec pod lib lint --verbose --fail-fast --swift-version=5.9
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

@dfed dfed merged commit 0b6dbe1 into main Nov 29, 2023
@dfed dfed deleted the dfed--swift-5-9 branch November 29, 2023 06:34
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