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

chore: merge master into release branch #586

Closed
wants to merge 8 commits into from
Closed

Conversation

robert-zaremba
Copy link
Collaborator

@robert-zaremba robert-zaremba commented Oct 19, 2021

Description

Merge master into the release branch. This merge will

  • update the Go to v1.17
  • add more tests
  • add script for starting a local Regen test node
  • add more documentation
  • fix: ORM IndexKeyCodec

Ref: #575


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • included the correct type prefix in the PR title
  • added ! to the type prefix if API or client breaking change
  • targeted the correct branch (see PR Targeting)
  • provided a link to the relevant issue or specification
  • followed the guidelines for building modules
  • included the necessary unit and integration tests
  • added a changelog entry to CHANGELOG.md
  • included comments for documenting Go code
  • updated the relevant documentation or specification
  • reviewed "Files changed" and left comments if necessary
  • confirmed all CI checks have passed

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.

I have...

  • confirmed the correct type prefix in the PR title
  • confirmed ! in the type prefix if API or client breaking change
  • confirmed all author checklist items have been addressed
  • reviewed state machine logic
  • reviewed API design and naming
  • reviewed documentation is accurate
  • reviewed tests and test coverage
  • manually tested (if applicable)

aleem1314 and others added 8 commits September 29, 2021 21:01
* feat: add weighted operations for update credit class msgs

* chore: fix typo

* chore: address review changes
* feat: WIP add grpc rest tests

* wip

* WIP adding tests

* add query tests

* fix failing test

* Update x/ecocredit/client/testsuite/grpc.go

Co-authored-by: Ryan Christoffersen <12519942+ryanchristo@users.noreply.github.com>

* Update x/ecocredit/client/testsuite/grpc.go

Co-authored-by: Ryan Christoffersen <12519942+ryanchristo@users.noreply.github.com>

* Update x/ecocredit/client/testsuite/grpc.go

Co-authored-by: Ryan Christoffersen <12519942+ryanchristo@users.noreply.github.com>

* update query batches endpoint

* Update x/ecocredit/client/testsuite/grpc.go

* run make proto-swagger-gen

* chore: fix errors

* fix failing tests

Co-authored-by: Ryan Christoffersen <12519942+ryanchristo@users.noreply.github.com>
Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com>
…ial (#538)

* added script to create a test node based on the tutorial instructions

* Apply suggestions from code review

I'm going to make one more change after this commit.

Co-authored-by: Ryan Christoffersen <12519942+ryanchristo@users.noreply.github.com>

* run the uname command once and assign to variable

* using bash shell rather than sh

Co-authored-by: Ryan Christoffersen <12519942+ryanchristo@users.noreply.github.com>
* docs: ecocredit client spec

* add update commands

* address review comment

* Update x/ecocredit/spec/05_client.md

Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com>

Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com>
* WIP Max255DynamicLengthIndexKeyCodec

* Add PrefixSearchableKey and get all tests working

* WIP changes to IndexKeyCodec

* wip on key codec

* Fix some tests

* Fix remaining tests

* Rm uint64_index

* Clean up and add more tests

* Update x/group and x/ecocredit

* Verify index key type

* Address some review comments

* Fix docs

* Add docs about NewIndex

Co-authored-by: blushi <marie.gauthier63@gmail.com>
Co-authored-by: likhita-809 <78951027+likhita-809@users.noreply.github.com>
* docs: update regen test networks

* docs: update regen test networks

* Update docs/getting-started/running-a-validator.md

Co-authored-by: likhita-809 <78951027+likhita-809@users.noreply.github.com>

* fix link

* minor revisions

* Update docs/getting-started/live-networks.md

Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com>

* fix link

Co-authored-by: likhita-809 <78951027+likhita-809@users.noreply.github.com>
Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com>
* chore: update cosmos sdk dependency

* update cosmos-sdk dependency in all nested modules
@codecov
Copy link

codecov bot commented Oct 19, 2021

Codecov Report

Merging #586 (6759708) into release/v2.0.x (6d17f39) will decrease coverage by 0.25%.
The diff coverage is 69.29%.

@@                Coverage Diff                 @@
##           release/v2.0.x     #586      +/-   ##
==================================================
- Coverage           75.94%   75.68%   -0.26%     
==================================================
  Files                 105      103       -2     
  Lines               13829    14204     +375     
==================================================
+ Hits                10502    10750     +248     
- Misses               2709     2808      +99     
- Partials              618      646      +28     
Flag Coverage Δ
experimental-codecov 75.68% <69.29%> (-0.26%) ⬇️
stable-codecov 67.19% <69.29%> (-0.12%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@robert-zaremba robert-zaremba changed the title chore: bump Go to v1.17 chore: merge master into release branch Oct 19, 2021
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.

5 participants