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

Change behaviors of create client, connection, and channel commands. #138

Merged
merged 9 commits into from
Apr 26, 2024

Conversation

dai1975
Copy link
Contributor

@dai1975 dai1975 commented Apr 13, 2024

  • just exit with status 1 if config.json has id but that client/connection/channel is not exists.
  • just exit with status 0 if config.json has id and that client/connection/channel is already opened.

@dai1975 dai1975 requested a review from a team as a code owner April 13, 2024 18:44
 - just exit with status 1 if config.json has id but that
   client/connection/channel is not exists.
 - just exit with status 0 if config.json has id and that
   client/connection/channel is already opened.

Signed-off-by: Daisuke Kanda <[email protected]>
Copy link
Contributor

@siburu siburu left a comment

Choose a reason for hiding this comment

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

@dai1975 Thank you. I have put some comments. Please check them and fix the code.

core/client.go Show resolved Hide resolved
core/client.go Outdated Show resolved Hide resolved
core/connection.go Outdated Show resolved Hide resolved
core/connection.go Outdated Show resolved Hide resolved
core/channel.go Outdated Show resolved Hide resolved
core/connection.go Outdated Show resolved Hide resolved
tests/cases/tm2tm/tests/Makefile Outdated Show resolved Hide resolved
tests/cases/tm2tm/tests/Makefile Outdated Show resolved Hide resolved
Signed-off-by: Daisuke Kanda <[email protected]>
@dai1975 dai1975 requested a review from siburu April 17, 2024 07:40
.github/workflows/test.yml Outdated Show resolved Hide resolved
core/client.go Outdated Show resolved Hide resolved
core/connection.go Outdated Show resolved Hide resolved
core/channel.go Outdated Show resolved Hide resolved
core/channel.go Outdated Show resolved Hide resolved
dai1975 added 5 commits April 19, 2024 08:56
Signed-off-by: Daisuke Kanda <[email protected]>
Signed-off-by: Daisuke Kanda <[email protected]>
Signed-off-by: Daisuke Kanda <[email protected]>
@dai1975 dai1975 requested a review from siburu April 19, 2024 09:56
Copy link
Contributor

@siburu siburu left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you!

@siburu siburu merged commit c00fc08 into hyperledger-labs:main Apr 26, 2024
6 checks passed
@dai1975 dai1975 deleted the pr/id-in-config branch April 26, 2024 06:22
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.

2 participants