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

Develop rebase with main branch #83

Merged
merged 5 commits into from Feb 17, 2022
Merged

Develop rebase with main branch #83

merged 5 commits into from Feb 17, 2022

Conversation

baziorek
Copy link
Member

In iroha-python library many newest merges was being merged into main branch instead of develop, that is why many changes are not available on develop.
So I've made:

git cherry-pick 41a724b3f3f21921ad455d39fb959f45c98fe986

to fetch: 41a724b

it will contain commits which change iroha/iroha.py file:

  1. 41a724b
  2. 92d8177
  3. 4d10eb0
  4. 3968c39

Rest of commits are not cherry-picked (connected with documentation, Jenkins, etc)

* Add ordering sequence to params

Makes possible to add ordering sequence to Query. Fixes #72

Signed-off-by: Rafik Naccache <rafik@fekr.tech>

* Add :param ordering_sequence:

Signed-off-by: Rafik Naccache <rafik@fekr.tech>

* Fix wrong example for :param ordering_sequence:

Signed-off-by: Rafik Naccache <rafik@fekr.tech>

* fix wrong ordering message construction in query

After more documentation I got to the way to construct to ordering object

Signed-off-by: Rafik Naccache <rafik@fekr.tech>

* remove redundant OR clause to manage ordering_sequence

Signed-off-by: Rafik Naccache <rafik@fekr.tech>
@baziorek
Copy link
Member Author

@Pawlak00 could You please take a look if I haven't damaged any Your changes?

iroha/iroha.py Outdated Show resolved Hide resolved
@baziorek baziorek marked this pull request as ready for review September 13, 2021 11:35
Signed-off-by: Grzegorz Bazior <bazior@agh.edu.pl>
Signed-off-by: G.Bazior <bazior@agh.edu.pl>
Signed-off-by: G.Bazior <bazior@agh.edu.pl>
@baziorek
Copy link
Member Author

baziorek commented Dec 21, 2021

@LiraLemur @iceseer I've just added more things to the PR:

  1. changes in protobuf files - it was done with attached script scripts/download-schema.py
  2. updated auto-generated protobuf files - it was done by script scripts/compile-proto.py - You can ignore that files in review (iroha/*_pb2.py)
  3. Two examples: examples/tls-example.py and examples/tx-example.py
  4. Others:
    1. setup.py and dev-requirements.txt - updated versions of dependencies
    2. SECURITY.md - copied from main branch

EDIT: And I've tested those changes by manually running script from: #72 (comment)

@baziorek
Copy link
Member Author

baziorek commented Feb 16, 2022

@iceseer Thanks for approve! So I'm merging its:).

EDIT: It looks like I don't have permissions, so I'll ask @LiraLemur

@baziorek baziorek merged commit d5e8cf4 into hyperledger:develop Feb 17, 2022
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.

None yet

4 participants