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.

Sign up for GitHub

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

Jump to bottom

Test release #792

Open
wants to merge 75 commits into
base: main
Choose a base branch
from
Open

Test release #792

wants to merge 75 commits into from

Conversation

DeepakBomjan
Copy link
Collaborator

Description

Commit Message

type: commit message

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.

viveksharmapoudel and others added 30 commits September 15, 2023 19:29
@viveksharmapoudel
fix: ibc query method get consensus state by height should return any…
b4d28eb 
… type (icon-project#727)

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

* fix: add uniformity
@redlarva
feat: remove xcall multi submodule (icon-project#733)
3c147a7 
* fix: remove xcall submodule

* refactor: update test execution script

* fix: update documentation
@izyak @viveksharmapoudel @nightowl121
feat: save write ack event height (icon-project#734)
6eb6a85 
* feat: save write ack event height

* chore: style fixes

Signed-off-by: Night Owl <nightowl121@protonmail.com>

* fix: add test case

---------

Signed-off-by: Night Owl <nightowl121@protonmail.com>
Co-authored-by: viveksharmapoudel <viveksharmapoudel@gmail.com>
Co-authored-by: Night Owl <nightowl121@protonmail.com>
@redlarva @debendraoli
@nightowl121 @DeepakBomjan
feat: test multiple packets on same height (icon-project#736)
26bcb55 
* add(test): multiple packets on same heigt

* rf: simplfy recursion call

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

Signed-off-by: Night Owl <nightowl121@protonmail.com>

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

---------

Signed-off-by: Night Owl <nightowl121@protonmail.com>
Co-authored-by: Debendra Oli <debendraoli@pm.me>
Co-authored-by: Night Owl <91905208+nightowl121@users.noreply.github.com>
Co-authored-by: DeepakBomjan <44976635+DeepakBomjan@users.noreply.github.com>
@ibrizsabin @sabinchitrakar
feat: add write ack log (icon-project#700)
30566dc 
* feat: add write ack log

* chore: pass build

---------

Co-authored-by: sabinchitrakar <immortal.infidel@gmail.com>
@ibrizsabin @redlarva
fix: point to xcall tag (icon-project#738)
22a2095 
* fix: point to xcall tag

* fix: xcall contract build script

---------

Co-authored-by: red__larva <red.x.larva@proton.me>
@redlarva
fix: address decoding on fee claim (icon-project#741)
ac4e4c7
@izyak @ibrizsabin
fix: do not send fund to ibc core on send_call_message (icon-project#740
e6574c6 
)

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

* fix: unit test

---------

Co-authored-by: ibrizsabin <sabin.chitrakar@ibriz.ai>
@DeepakBomjan @ibrizsabin
feat: Add artifact checksum in release (icon-project#742)
d91fd53 
* chore: pass build

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

---------

Co-authored-by: ibrizsabin <sabin.chitrakar@ibriz.ai>
@DeepakBomjan
ci: fix download error (icon-project#744)
587fb08
@DeepakBomjan
ci: revert release workflow changes (icon-project#746)
b0f9956 
* ci: debug

* ci: debug

* ci: revert release workflow changes
@DeepakBomjan
ci: cleanup
e4fd064
@DeepakBomjan
ci: add deployment env setup job
3b619aa
@DeepakBomjan
ci: add deployment env setup job
182d149
@DeepakBomjan
ci: add deployment env setup job
10245f7
@DeepakBomjan
test
00203be
@DeepakBomjan
ci:update action
2df613b
@DeepakBomjan
ci:update action
d96b23e
@DeepakBomjan
ci: test
3af0c2d
@DeepakBomjan
Merge pull request #1 from DeepakBomjan/fix-key-management
2d962e7 
Fix key management
@DeepakBomjan
ci: test
3eb0082
@DeepakBomjan
ci: test
63523eb
@DeepakBomjan
ci: test
ac1fa6f
@DeepakBomjan
Merge pull request #2 from DeepakBomjan/fix-key-management
d67cf3b 
Fix key management
@DeepakBomjan
ci: fix input var
85f7a53
@DeepakBomjan
ci: fix input var
7a1541e
@DeepakBomjan
ci: fix input var
7f249f7
@DeepakBomjan
ci: test
a9f0993
@DeepakBomjan
ci:debug
9a03210
@DeepakBomjan
ci:debug
525a9af
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@redlarva redlarva Awaiting requested review from redlarva

@ibrizsabin ibrizsabin Awaiting requested review from ibrizsabin ibrizsabin is a code owner

@AntonAndell AntonAndell Awaiting requested review from AntonAndell AntonAndell is a code owner

@nightowl121 nightowl121 Awaiting requested review from nightowl121 nightowl121 is a code owner

@sdpisreddevil sdpisreddevil Awaiting requested review from sdpisreddevil sdpisreddevil is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@DeepakBomjan @viveksharmapoudel @redlarva @izyak @ibrizsabin