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

auto_operation にprotobuf の型ファイルを導入 #263

Merged
merged 17 commits into from
Nov 15, 2022

Conversation

Azuki-bar
Copy link
Member

@Azuki-bar Azuki-bar commented Nov 15, 2022

pyiファイルを生成させ、mypyで検査するつもりです

mypyでのチェックを合格させるには型アノテーションをつけてまわる仕事をしないといけないので取りあえずCIによるチェックは後回しにします

ついでにCIでフォーマッタも導入しました

Azuki-bar and others added 3 commits November 15, 2022 14:28

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@github-actions github-actions bot added the ci-cd label Nov 15, 2022
@Azuki-bar Azuki-bar changed the title auto_operation にgrpc の型チェックを導入 auto_operation にprotobuf の型ファイルを導入 Nov 15, 2022
@Azuki-bar Azuki-bar requested review from csenet and n4o847 November 15, 2022 06:25
@n4o847
Copy link
Collaborator

n4o847 commented Nov 15, 2022

ちょうど型定義がほしいと思っていたため、助かります……!

auto_operation 本体の型付けは、今後やっていきます

@Azuki-bar
Copy link
Member Author

@n4o847
現在、mypyで50件くらい怒られています
ですが、mypyは型推論してアノテートすべき型を提示してくれているのでこれを貼っていけばなんとかなると思います

https://github.com/ueckoken/plarail2022/actions/runs/3467966675/jobs/5793328370#step:5:13

@Azuki-bar Azuki-bar requested a review from n4o847 November 15, 2022 06:46
@Azuki-bar
Copy link
Member Author

@n4o847 CIでフォーマッタも導入しました
レビューのApproveを受けてから結構な変更が入りましたので確認お願いします

@n4o847
Copy link
Collaborator

n4o847 commented Nov 15, 2022

型とフォーマットの整備ありがとうございます…… 🙏

CI 環境など、今後プラレーラーズの方の開発でも参考にさせていただきます

@thgcMtdh も今ブランチ立ててコード書いてると思うので、確認しておくといいと思います

@Azuki-bar Azuki-bar merged commit 18c25a5 into main Nov 15, 2022
@Azuki-bar Azuki-bar deleted the auto-operation-type-check branch November 15, 2022 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants