Skip to content

Conversation

tera-ny
Copy link
Contributor

@tera-ny tera-ny commented Jul 31, 2025

Swift6を有効化して出たエラー、警告を修正しました。

@tera-ny tera-ny self-assigned this Jul 31, 2025
@tera-ny
Copy link
Contributor Author

tera-ny commented Jul 31, 2025

CI周りも修正する必要があるのか、、

@tera-ny tera-ny requested a review from onoyuuya July 31, 2025 06:30
@tera-ny tera-ny force-pushed the tera-ny/migrate_swift6 branch from f4afe9d to e030f97 Compare August 29, 2025 03:21
DEVELOPER_DIR: '/Applications/Xcode_${{ matrix.xcode }}.app/Contents/Developer'
strategy:
matrix:
xcode: [ 15.4, 16.4 ]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Swift-tools-version:6.0 is not supported in Xcode 15.4, so we are ending support for Xcode 15.4 in CI.


var changes: [UserDefaults.Change<String>] = []
let observer = wrapper.addObserver { changes.append($0) }
addTeardownBlock(observer.invalidate)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Since it seems unnecessary to use addTeardownBlock, I removed it.

var observer: UserDefaults.Observation? = userDefaults.observeObject(forKey: "TestKey") { change in
changes.append(change)
}
_ = observer
Copy link
Member

Choose a reason for hiding this comment

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

Q. これは必要なコードですか

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Linterエラー対策で追記しました

@tera-ny tera-ny force-pushed the tera-ny/migrate_swift6 branch from fea83a9 to d4a0c33 Compare August 29, 2025 05:13
@tera-ny
Copy link
Contributor Author

tera-ny commented Aug 29, 2025

マージします

@tera-ny tera-ny merged commit d27c189 into main Aug 29, 2025
6 checks passed
@tera-ny tera-ny deleted the tera-ny/migrate_swift6 branch August 29, 2025 05:35
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.

2 participants