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

デリゲート課題の順番を移動しました。 #51

Merged

Conversation

es-kumagai
Copy link
Contributor

A. デリゲートの順番変更

デリゲート課題の登場順番は迷うところなのですが、育成 WG で相談して、今回は 非同期処理 の課題の一環としてデリゲートを扱うことにするため、その課題の登場位置を ThreadBlockConcurrency までのグループ内に移動しました。

B. デリゲート課題の微調整

また、以前にデリゲート課題を移動した際に、その課題内で 同期 API を使って という記載を API を使って に直しましたが、同期 API を使うことに限定した方が ThreadBlock の次の課題として良い学習ができそうに思えたため、文言を元に戻してあります。

C. 課題名称の変更

これまで Closure として用意されていた課題を、その学習目的が「非同期処理」であることに合わせて Callback に変更しました。

D. セッションのグループ分け

加えて、課題を進める際の心持ちが定まるように、セッションをグループ分けして README.md に記載しました。各セクションの番号が整数で連番になるように整えてあります。この変更については今回思い立って実施したため、適切かも含めて必要か相談したいところです。

Delegate の登場順番は悩ましいところですが、非同期処理の課題を束ね、複数の方法を比べながら学習を進められることに重きを置くことにしました。

Revert "Merge pull request yumemi-inc#27 from yumemi-inc/feature/move_delegate_section"

This reverts commit d2d07f7, reversing
changes made to 0c5dc57.
@ykws ykws added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 27, 2023
Copy link
Member

@ykws ykws left a comment

Choose a reason for hiding this comment

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

変更の方向性は LGTM です。
表現の統一に関してコメントしたので確認をお願いしたいです。

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Documentation/Delegate.md Outdated Show resolved Hide resolved
表現を "アプリ" で統一しました。

Co-authored-by: KAWASHIMA Yoshiyuki <ykawashi7@gmail.com>
@es-kumagai
Copy link
Contributor Author

素敵なレビューをありがとうございます。まずはさっそく "App" を "アプリ" に表現統一しました。

@novr novr requested a review from a team November 6, 2023 06:39
@novr novr requested a review from a team November 6, 2023 06:41
Copy link
Member

@el-hoshino el-hoshino left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: KAWASHIMA Yoshiyuki <ykawashi7@gmail.com>
Copy link
Member

@ykws ykws left a comment

Choose a reason for hiding this comment

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

LGTM 👍

順番が変わるので、バージョンを変更してリリースしても良さそうに思いました。
現在課題に取り組んでいる人もいるので、どのバージョンで進めているかで認識合わせしやすくなると思います。
現在 2.1.0 なので、 2.2.0 くらいですかね。

@es-kumagai es-kumagai merged commit 6f2de84 into yumemi-inc:main Dec 10, 2023
2 checks passed
ykws added a commit to ykws/ios-training that referenced this pull request Dec 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants