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

YumemiWeather を iOS 13 以降にしてはどうか。 #74

Open
yumemi-kumagai opened this issue Apr 30, 2024 · 1 comment
Open

YumemiWeather を iOS 13 以降にしてはどうか。 #74

yumemi-kumagai opened this issue Apr 30, 2024 · 1 comment

Comments

@yumemi-kumagai
Copy link
Contributor

YumemiWeather の API の一部に iOS 13 以降で有効と明記されたものがいくつかあります。

たとえば、非同期で天気情報を拾ってくるメソッドがそれに該当します。

@available(iOS 13, macOS 10.15, *)
    public static func asyncFetchWeather(_ jsonString: String) async throws -> String {

これが使えなくても同期版があるため問題はないのですけれど、課題の環境条件に「基本的に最新の安定版を利用すること」とあるのと、バグ解消の課題で使う Example プロジェクトが iOS 17.2 を求めているのを踏まえると、YumemiWeather で一部の機能を iOS 13 以降で解禁するよりも、YumemiWeather 自体を iOS 13 以降にするのが自然かもしれません。

@yumemi-kumagai
Copy link
Contributor Author

非同期版の使用を求める課題が存在しないこと、そして同期版さえあれば非同期処理を実装できること(課題でもある)を踏まえると、むしろ既存の非同期版 API を廃止する手もありそうです。

iOS 13 以降に制限されているものは非同期に関するものなので、それによって当 Issue を解消できます。

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

No branches or pull requests

1 participant