Skip to content

同期的に実行できるように#938

Merged
syuilo merged 6 commits intomasterfrom
sync
Aug 4, 2025
Merged

同期的に実行できるように#938
syuilo merged 6 commits intomasterfrom
sync

Conversation

@syuilo
Copy link
Collaborator

@syuilo syuilo commented Jul 14, 2025

What

Resolve #937

Why

Additional info (optional)

@syuilo
Copy link
Collaborator Author

syuilo commented Jul 14, 2025

ほぼ同じコードが二重に存在するの無駄感が強いけど他に良い実装方法が浮かばなかった

@anatawa12
Copy link

anatawa12 commented Jul 14, 2025

ほぼ同じコードが二重に存在するの無駄感が強いけど他に良い実装方法が浮かばなかった

一応 generator で頑張るって手がなくはないかもですが、きれいなコードにはあまりならなさそう

@anatawa12
Copy link

👀 がついたので参考文献を一つ貼っておきます https://speakerdeck.com/susisu/tskaigi-2025

@syuilo syuilo mentioned this pull request Jul 15, 2025
4 tasks
@syuilo syuilo marked this pull request as ready for review August 1, 2025 08:38
@syuilo
Copy link
Collaborator Author

syuilo commented Aug 4, 2025

マージするか

@syuilo syuilo merged commit 71f463e into master Aug 4, 2025
4 checks passed
@syuilo syuilo deleted the sync branch August 4, 2025 01:54
@syuilo
Copy link
Collaborator Author

syuilo commented Aug 4, 2025

🙏🏻

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.

sync版execメソッド

2 participants