-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat: biyooon, adding using zenohex #7
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
こちらも概ねおkです.所々冗長だったり不要なものがまだ混じっているようなので精査をお願いしまう
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
私としては LGTM!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fomfom 全般的にコメントの書き方が不安定な印象なので修正してもらいたいです
https://qiita.com/jnchito/items/f0d90af4ed44b7484103
lib/giocci_engine_zenoh.ex
Outdated
@doc """ | ||
最初に指定されたRelayノードとのZenohコネクションを作成する | ||
""" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@docがどこにかかってるか分かりづらいので改行のけましょう(他にも多数)
@doc """
docdocdoc...
"""
def hoge() do
:hoge
end
lib/giocci_engine_zenoh.ex
Outdated
engine_name = Application.get_env(:giocci_engine_zenoh, :system_variables)[:my_node_name] | ||
|
||
{:ok, session} = Zenohex.open() | ||
## pub,subそれぞれのキーをたてる |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fomfom どこの処理に対するコメントが分かりづらいので修正お願いします
一部の従来の機能に通信部分をZenohでおきかえたverを追加したものです