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: merge release #759

Merged
merged 13 commits into from
Oct 4, 2023
Merged

fix: merge release #759

merged 13 commits into from
Oct 4, 2023

Conversation

ibrizsabin
Copy link
Collaborator

@ibrizsabin ibrizsabin commented Sep 29, 2023

Description

Commit Message

fix: merge release

see the guidelines for commit messages.

Changelog Entry

version: <log entry>

Checklist

  • I have performed a self-review of my own code
  • I have documented my code in accordance with the documentation guidelines
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the unit tests
  • I only have one commit (if not, squash them into one commit).
  • I have a descriptive commit message that adheres to the commit message guidelines
  • I have added version bump label on PR.

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

nightowl121 and others added 13 commits September 11, 2023 17:55
* feat/cw-mock-ibc-connection added

* feat: mock xcall connection added

* fix: cargo lints fixes

* fix: mock xcall connection functionality reduced

* fix: cargo lints fixes
* fix: funds compare issue

* fix: fee check added in sendMessage

* style: caro lints fixes

* feat: unit test for funds in send message added

* chore: cargo lints fixes

* fix: funds compare issue

* fix: fee check added in sendMessage

* style: caro lints fixes

* feat: unit test for funds in send message added

* chore: cargo lints fixes

* fix: bug fixed in send message funds test

* fix: cargo lints fixes
…ixed (#687)

* admin and owner state changed to Addr, and corresponding tests fixed

* style: caro lints fixes

* Revert "style: caro lints fixes"

This reverts commit 755c628.

* fix: cargo lints fixes

---------

Co-authored-by: ibrizsabin <[email protected]>
* fix: remove_admin and update_owner is removed, and created a method for update_admin

* fix: transfer admin removed and add_admin updated

* fix: transfer admin removed and add_admin updated

* fix: add_owner_unauthorized removed
* ci: add terraform deployment script

* ci: update instance tag value

* ci: fix goloop command not found by adding goloop bin path in sudo secure path

* ci: cleanup deployment script

* ci: restrict keystore directory

* ci: update permission
* ci: make secret files inaccessable to other users

* ci: fix permission
@github-actions github-actions bot added the cicd label Sep 29, 2023
@ibrizsabin ibrizsabin added bug Something isn't working and removed cicd labels Sep 29, 2023
@github-actions github-actions bot added the cicd label Sep 29, 2023
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (release@a241bc6). Click here to learn what that means.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             release     #759   +/-   ##
==========================================
  Coverage           ?   70.01%           
  Complexity         ?      398           
==========================================
  Files              ?      154           
  Lines              ?    13901           
  Branches           ?      287           
==========================================
  Hits               ?     9733           
  Misses             ?     4016           
  Partials           ?      152           
Flag Coverage Δ
rust 68.22% <0.00%> (?)

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

Copy link
Collaborator

@redlarva redlarva left a comment

Choose a reason for hiding this comment

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

looks good

@redlarva redlarva merged commit 6b3e4fc into release Oct 4, 2023
@redlarva redlarva deleted the fix/merge-release branch October 4, 2023 04:19
redlarva added a commit that referenced this pull request Oct 4, 2023
* fix: ibc query method get consensus state by height should return any type (#727)

* patch: ibc query method get consensus state by height should return any type

* fix: add uniformity

* feat: remove xcall multi submodule (#733)

* fix: remove xcall submodule

* refactor: update test execution script

* fix: update documentation

* feat: save write ack event height (#734)

* feat: save write ack event height

* chore: style fixes

Signed-off-by: Night Owl <[email protected]>

* fix: add test case

---------

Signed-off-by: Night Owl <[email protected]>
Co-authored-by: viveksharmapoudel <[email protected]>
Co-authored-by: Night Owl <[email protected]>

* feat: test multiple packets on same height (#736)

* add(test): multiple packets on same heigt

* rf: simplfy recursion call

* ci: remove cw_common from inside the docker container (#712)

Signed-off-by: Night Owl <[email protected]>

* chore: exclude cosmwasm library and common file from contract deployment (#698)

---------

Signed-off-by: Night Owl <[email protected]>
Co-authored-by: Debendra Oli <[email protected]>
Co-authored-by: Night Owl <[email protected]>
Co-authored-by: DeepakBomjan <[email protected]>

* feat: add write ack log (#700)

* feat: add write ack log

* chore: pass build

---------

Co-authored-by: sabinchitrakar <[email protected]>

* fix: point to xcall tag (#738)

* fix: point to xcall tag

* fix: xcall contract build script

---------

Co-authored-by: red__larva <[email protected]>

* fix: address decoding on fee claim (#741)

* fix: do not send fund to ibc core on send_call_message (#740)

* fix: do not send fund to ibc host on send call message

* fix: unit test

---------

Co-authored-by: ibrizsabin <[email protected]>

* feat: Add artifact checksum in release (#742)

* chore: pass build

* ci: cleanup release artifacts upload
* dynamically fetch wasm version
* chore: cleanup redundant steps from the workflow

---------

Co-authored-by: ibrizsabin <[email protected]>

* ci: fix download error (#744)

* ci: revert release workflow changes (#746)

* ci: debug

* ci: debug

* ci: revert release workflow changes

* fix: call update admin (#751)

* fix: call-update-admin

* chore: pass build

* ci: update release build tag format (#758)

ci: trigger release action on any tag format and enable release

* fix: merge release (#759)

* ci: remove cw_common from inside the docker container (#712)

Signed-off-by: Night Owl <[email protected]>

* feat: cw-mock-ibc-connection added (#711)

* feat/cw-mock-ibc-connection added

* feat: mock xcall connection added

* fix: cargo lints fixes

* fix: mock xcall connection functionality reduced

* fix: cargo lints fixes

* chore: exclude cosmwasm library and common file from contract deployment (#698)

* fix: fee check added in sendMessage (#684)

* fix: funds compare issue

* fix: fee check added in sendMessage

* style: caro lints fixes

* feat: unit test for funds in send message added

* chore: cargo lints fixes

* fix: funds compare issue

* fix: fee check added in sendMessage

* style: caro lints fixes

* feat: unit test for funds in send message added

* chore: cargo lints fixes

* fix: bug fixed in send message funds test

* fix: cargo lints fixes

* fix: owner check removed in addAdmin in xcall-mock-connection (#715)

* fix: admin and owner state changed to Addr, and corresponding tests fixed (#687)

* admin and owner state changed to Addr, and corresponding tests fixed

* style: caro lints fixes

* Revert "style: caro lints fixes"

This reverts commit 755c628.

* fix: cargo lints fixes

---------

Co-authored-by: ibrizsabin <[email protected]>

* fix: remove_admin and update_owner, update_admin are removed (#692)

* fix: remove_admin and update_owner is removed, and created a method for update_admin

* fix: transfer admin removed and add_admin updated

* fix: transfer admin removed and add_admin updated

* fix: add_owner_unauthorized removed

* ci: add terraform deployment script (#750)

* ci: add terraform deployment script

* ci: update instance tag value

* ci: fix goloop command not found by adding goloop bin path in sudo secure path

* ci: cleanup deployment script

* ci: restrict keystore directory

* ci: update permission

* ci: make secret files inaccessable to other users (#753)

* ci: make secret files inaccessable to other users

* ci: fix permission

* ci: fix typo (#756)

* fix: update admin tests

* chore: pass build

---------

Signed-off-by: Night Owl <[email protected]>
Co-authored-by: Night Owl <[email protected]>
Co-authored-by: gcranju <[email protected]>
Co-authored-by: DeepakBomjan <[email protected]>

---------

Signed-off-by: Night Owl <[email protected]>
Co-authored-by: viveksharmapoudel <[email protected]>
Co-authored-by: redlarva <[email protected]>
Co-authored-by: izyak <[email protected]>
Co-authored-by: Night Owl <[email protected]>
Co-authored-by: Debendra Oli <[email protected]>
Co-authored-by: Night Owl <[email protected]>
Co-authored-by: DeepakBomjan <[email protected]>
Co-authored-by: sabinchitrakar <[email protected]>
Co-authored-by: red__larva <[email protected]>
Co-authored-by: gcranju <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cicd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants