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

fix variable error #4020

Merged
merged 5 commits into from
Jan 29, 2024
Merged

fix variable error #4020

merged 5 commits into from
Jan 29, 2024

Conversation

xiaolou86
Copy link
Contributor

@xiaolou86 xiaolou86 commented Oct 27, 2023

  1. fix variable error in example/burrow_integration/permissions.py
  2. fix some typos

CMakeLists.txt Outdated Show resolved Hide resolved
appetrosyan
appetrosyan previously approved these changes Jan 10, 2024
Signed-off-by: xiaolou86 <[email protected]>
Signed-off-by: xiaolou86 <[email protected]>
Co-authored-by: Aleksandr Petrosyan <[email protected]>
Signed-off-by: xiaolou86 <[email protected]>
@xiaolou86
Copy link
Contributor Author

xiaolou86 commented Jan 11, 2024

@appetrosyan Thanks for your review. I have added the Suggested change.

@xiaolou86 xiaolou86 requested a review from appetrosyan January 11, 2024 02:37
Copy link
Contributor

@baziorek baziorek left a comment

Choose a reason for hiding this comment

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

Thanks for correcting typos. Just two small changes to fix after renaming

example/burrow_integration/permissions.py Show resolved Hide resolved
example/burrow_integration/permissions.py Show resolved Hide resolved
@xiaolou86 xiaolou86 requested a review from baziorek January 11, 2024 07:12
@xiaolou86
Copy link
Contributor Author

@baziorek thank you very much for your review.

Copy link
Contributor

@baziorek baziorek left a comment

Choose a reason for hiding this comment

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

LGTM

@xiaolou86
Copy link
Contributor Author

@appetrosyan @baziorek thanks for your reviews. Could you help to merge?

@6r1d 6r1d merged commit ffed6b0 into hyperledger-iroha:main Jan 29, 2024
25 checks passed
@baziorek
Copy link
Contributor

@appetrosyan @baziorek thanks for your reviews. Could you help to merge?

It was merged already by @6r1d . Thanks @xiaolou86 for Your work!

dominious1 pushed a commit to dominious1/iroha that referenced this pull request Feb 1, 2024
* docs: fix typos

Signed-off-by: xiaolou86 <[email protected]>

* fix variable error

Signed-off-by: xiaolou86 <[email protected]>

* Fix typo in CMakeLists.txt

Co-authored-by: Aleksandr Petrosyan <[email protected]>
Signed-off-by: xiaolou86 <[email protected]>

* fix typo in docs/source/build/index.rst

Signed-off-by: xiaolou86 <[email protected]>

---------

Signed-off-by: xiaolou86 <[email protected]>
Co-authored-by: Aleksandr Petrosyan <[email protected]>
Signed-off-by: dominious1 <[email protected]>
@nxsaken nxsaken added the iroha1 The legacy version of Iroha. label Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.x iroha1 The legacy version of Iroha.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants