We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
https://github.com/moricho/tparallel/blob/master/tparallel.go#L15 いい感じのコメントを書いたほうが良さそう
https://github.com/moricho/tparallel/blob/master/tparallel.go#L34 非公開な型に依存すると将来壊れる可能性あるのでtypes.LookupFieldOrMethodを使った方がよさそう。 参考:https://docs.google.com/presentation/d/1I4pHnzV2dFOMbRcpA-XD0TaLcX6PBKpls6WxGHoMjOg/edit#slide=id.g80c1410104_5_250
types.LookupFieldOrMethod
https://github.com/moricho/tparallel/blob/master/tparallel.go#L43 よいコメント 👍
https://github.com/moricho/tparallel/blob/master/tparallel.go#L89 先頭の命令だけで良さそう
よくかけてて良いですね!!
The text was updated successfully, but these errors were encountered:
【コメント】 メルカリのテックブログでも話題になったt.Parallelがされているかの検出を行うツールを作ってくれました。コーナーケースを頑張って探して潰してくれているので、実用的なツールになってるんでは無いでしょうか。
Sorry, something went wrong.
No branches or pull requests
https://github.com/moricho/tparallel/blob/master/tparallel.go#L15
いい感じのコメントを書いたほうが良さそう
https://github.com/moricho/tparallel/blob/master/tparallel.go#L34
非公開な型に依存すると将来壊れる可能性あるので
types.LookupFieldOrMethod
を使った方がよさそう。参考:https://docs.google.com/presentation/d/1I4pHnzV2dFOMbRcpA-XD0TaLcX6PBKpls6WxGHoMjOg/edit#slide=id.g80c1410104_5_250
https://github.com/moricho/tparallel/blob/master/tparallel.go#L43
よいコメント 👍
https://github.com/moricho/tparallel/blob/master/tparallel.go#L89
先頭の命令だけで良さそう
よくかけてて良いですね!!
The text was updated successfully, but these errors were encountered: