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

App の initializer, entry_point の返り値を void から RESULT にする #19

Merged
merged 3 commits into from
Jul 20, 2023

Conversation

meltingrabbit
Copy link
Member

@meltingrabbit meltingrabbit commented Jul 19, 2023

概要

App の initializer, entry_point の返り値を void から RESULT にする

Issue

詳細

App の初期化,実行のエラーをAMに通知する方法を追加

検証結果

全てのテストが通った

影響範囲

user の App の initializer, entry_point を修正する必要がある

補足

@meltingrabbit meltingrabbit added the enhancement New feature or request label Jul 19, 2023
@meltingrabbit meltingrabbit self-assigned this Jul 19, 2023
@meltingrabbit meltingrabbit changed the base branch from develop to feature/fix_url_of_docs July 19, 2023 17:53
@meltingrabbit
Copy link
Member Author

@ToshiAki64 別の機能実装中に欲しくなったので機能追加してます.
レビューいただけますか?

レビュー後に,OKそうであれば,既存の App の関数の型を直していきます.

Copy link
Collaborator

@ToshiAki64 ToshiAki64 left a comment

Choose a reason for hiding this comment

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

いいと思います。

@meltingrabbit meltingrabbit changed the title WIP: App の initializer, entry_point の返り値を void から RESULT にする App の initializer, entry_point の返り値を void から RESULT にする Jul 20, 2023
@meltingrabbit
Copy link
Member Author

@ToshiAki64 ありがとうございます.
app の retrun の修正とテストを通すのが完了したため,再度レビューください.

Copy link
Collaborator

@ToshiAki64 ToshiAki64 left a comment

Choose a reason for hiding this comment

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

よさそうです。

Base automatically changed from feature/fix_url_of_docs to develop July 20, 2023 08:46
@meltingrabbit meltingrabbit force-pushed the feature/change_ret_type_of_app_initializer branch from 6521bc2 to 8164836 Compare July 20, 2023 08:47
@meltingrabbit meltingrabbit merged commit b592343 into develop Jul 20, 2023
24 checks passed
@meltingrabbit meltingrabbit deleted the feature/change_ret_type_of_app_initializer branch July 20, 2023 08:53
sksat added a commit that referenced this pull request Oct 17, 2023
sksat added a commit that referenced this pull request Oct 27, 2023
sksat added a commit that referenced this pull request Oct 30, 2023
sksat added a commit that referenced this pull request Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants