last nits

.coderabbit.yml

@@ -11,22 +11,22 @@ reviews:
         - "!docs/"
     path_instructions:
         - path: "**/*.go"
-          instructions: "Review the Golang code for conformity with the Uber Golang style guide, highlighting any deviations."
+          instructions: "Review the Golang code for conformity with the Uber Golang style guide, highlighting any deviations. Only report issues that you have a high degree of confidence in."
         - path: "tests/e2e/*"
           instructions: |
-              "Assess the e2e test code assessing sufficient code coverage for the changes associated in the pull request"
+              "Assess the e2e test code assessing sufficient code coverage for the changes associated in the pull request. Only report issues that you have a high degree of confidence in."
         - path: "tests/integration/*"
           instructions: |
-              "Assess the e2e test code assessing sufficient code coverage for the changes associated in the pull request"
+              "Assess the e2e test code assessing sufficient code coverage for the changes associated in the pull request. Only report issues that you have a high degree of confidence in."
         - path: "**/*_test.go"
           instructions: |
-              "Assess the unit test code assessing sufficient code coverage for the changes associated in the pull request"
+              "Assess the unit test code assessing sufficient code coverage for the changes associated in the pull request. Only report issues that you have a high degree of confidence in."
         - path: "**/*.md"
           instructions: |
-              "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness. Please DO NOT report any missing or superfluous newlines, in particular at the end or beginning of files."
+              "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness. Please DO NOT report any missing or superfluous newlines, in particular at the end or beginning of files. Only report issues that you have a high degree of confidence in."
         - path: ".changelog/*"
           instructions: |
-              "Assess the changes in the changelog for correctness and completeness, particularly flagging missing changes"
+              "Assess the changes in the changelog for correctness and completeness, particularly flagging missing changes. Only report issues that you have a high degree of confidence in."
     auto_review:
         enabled: true
         ignore_title_keywords:

.github/workflows/deploy-docs.yml

@@ -42,7 +42,7 @@ jobs:
           ./build_deploy.sh
 
       - name: Deploy 🚀
-        uses: JamesIves/[email protected].1
+        uses: JamesIves/[email protected].3
         with:
           branch: gh-pages
           folder: ~/output

.github/workflows/docker-publish.yml

@@ -50,7 +50,7 @@ jobs:
       # multi-platform images and export cache
       # https://github.com/docker/setup-buildx-action
       - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@d70bba72b1f3fd22344832f00baa16ece964efeb # v3.3.0
+        uses: docker/setup-buildx-action@4fd812986e6c8c2a69e18311145f9371337f27d4 # v3.4.0
 
 
       # Login against a Docker registry except on PR
@@ -80,7 +80,7 @@ jobs:
       # https://github.com/docker/build-push-action
       - name: Build and push Docker image
         id: build-and-push
-        uses: docker/build-push-action@15560696de535e4014efeff63c48f16952e52dd1 #v6.2.0
+        uses: docker/build-push-action@1a162644f9a7e87d8f4b053101d1d9a712edc18c #v6.3.0
         with:
           context: .
           file: ./Dockerfile

CHANGELOG.md

@@ -39,6 +39,25 @@
 - Revert `PutUnbondingOnHold` behavior to ICS@v1
 ([\#1819](https://github.com/cosmos/interchain-security/pull/1819))
 
+## v4.4.0
+
+*July 16, 2024*
+
+### API BREAKING
+
+- Remove soft opt-out feature. 
+  ([\#1964](https://github.com/cosmos/interchain-security/pull/1964))
+
+### FEATURES
+
+- Remove soft opt-out feature. 
+  ([\#1964](https://github.com/cosmos/interchain-security/pull/1964))
+
+### STATE BREAKING
+
+- Remove soft opt-out feature. 
+  ([\#1964](https://github.com/cosmos/interchain-security/pull/1964))
+
 ## v4.3.1
 
 *July 4, 2024*

UPGRADING.md

@@ -1,7 +1,38 @@
 # Upgrading Replicated Security
 
+## [v4.4.x](https://github.com/cosmos/interchain-security/releases/tag/v4.4.0)
+
+### Provider
+
+***Note that provider chains should not use this version of ICS***
+
+### Consumer
+
+Upgrading the consumer from `v4.0.0` to `v4.4.0` will not require state migration. 
+
 This guide provides instructions for upgrading to specific versions of Replicated Security.
 
+## [v4.3.x](https://github.com/cosmos/interchain-security/releases/tag/v4.3.0)
+
+### Provider
+
+Upgrading a provider from `v4.2.0` to `v4.3.0` requires state migrations that will be done automatically via the upgrade module. 
+
+### Consumer
+
+***Note that consumer chains should not use this version of ICS***
+
+## [v4.2.x](https://github.com/cosmos/interchain-security/releases/tag/v4.2.0)
+
+### Provider
+
+Upgrading a provider from `v4.1.0` or `v4.1.0-lsm` to `v4.2.0` or `v4.2.0-lsm` requires state migrations, see relevant pull request [here](https://github.com/cosmos/interchain-security/pull/1809)
+for the corresponding migrators.
+
+### Consumer
+
+***Note that consumer chains should not use this version of ICS***
+
 ## [v4.1.x](https://github.com/cosmos/interchain-security/releases/tag/v4.1.0-rc2) and [v4.1.x-lsm](https://github.com/cosmos/interchain-security/releases/tag/v4.1.0-lsm-rc2)
 
 ### Provider
@@ -80,7 +111,7 @@ Upgrading the provider from `v2.x.y` to `v3.3.0` will not require state migratio
 
 ## [v3.2.x](https://github.com/cosmos/interchain-security/tree/release/v3.2.x)
 
-`v3.2.0` bumps IBC to `v7.3`. As a result, `legacy_ibc_testing` is no longer required and was removed, see https://github.com/cosmos/interchain-security/pull/1185. This means that when upgrading to `v3.2.0`, any customized tests relying on `legacy_ibc_testing` need to be updated.
+`v3.2.0` bumps IBC to `v7.3`. As a result, `legacy_ibc_testing` is not longer required and was removed, see https://github.com/cosmos/interchain-security/pull/1185. This means that when upgrading to `v3.2.0`, any customized tests relying on `legacy_ibc_testing` need to be updated.
 
 ### Consumer 
 
@@ -124,7 +155,7 @@ Legacy APIs of the `AppModule` interface have been removed from ccv modules. For
 
 #### Imports
 
-Imports for ics23 have been updated as the repository has been migrated from confio to cosmos.
+Imports for ics23 have been updated as the repository have been migrated from confio to cosmos.
 
 ```diff
 import (

go.mod

@@ -72,7 +72,7 @@ require (
 	github.com/godbus/dbus v0.0.0-20190726142602-4481cbc300e2 // indirect
 	github.com/gogo/googleapis v1.4.1 // indirect
 	github.com/gogo/protobuf v1.3.2 // indirect
-	github.com/golang/glog v1.2.0 // indirect
+	github.com/golang/glog v1.2.1 // indirect
 	github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da // indirect
 	github.com/golang/snappy v0.0.4 // indirect
 	github.com/google/btree v1.1.2 // indirect
@@ -138,7 +138,6 @@ require (
 	golang.org/x/term v0.19.0 // indirect
 	golang.org/x/text v0.14.0 // indirect
 	google.golang.org/api v0.171.0 // indirect
-	google.golang.org/appengine v1.6.8 // indirect
 	gopkg.in/ini.v1 v1.67.0 // indirect
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 	nhooyr.io/websocket v1.8.6 // indirect
@@ -210,6 +209,7 @@ require (
 	go.uber.org/multierr v1.10.0 // indirect
 	golang.org/x/sync v0.7.0 // indirect
 	golang.org/x/time v0.5.0 // indirect
+	google.golang.org/appengine v1.6.8 // indirect
 	google.golang.org/genproto/googleapis/rpc v0.0.0-20240401170217-c3f982113cda // indirect
 	gotest.tools/v3 v3.5.1 // indirect
 )

go.sum

@@ -501,8 +501,8 @@ github.com/gogo/protobuf v1.3.1/go.mod h1:SlYgWuQ5SjCEi6WLHjHCa1yvBfUnHcTbrrZtXP
 github.com/gogo/protobuf v1.3.2 h1:Ov1cvc58UF3b5XjBnZv7+opcTcQFZebYjWzi34vdm4Q=
 github.com/gogo/protobuf v1.3.2/go.mod h1:P1XiOD3dCwIKUDQYPy72D8LYyHL2YPYrpS2s69NZV8Q=
 github.com/golang/glog v0.0.0-20160126235308-23def4e6c14b/go.mod h1:SBH7ygxi8pfUlaOkMMuAQtPIUF8ecWP5IEl/CR7VP2Q=
-github.com/golang/glog v1.2.0 h1:uCdmnmatrKCgMBlM4rMuJZWOkPDqdbZPnrMXDY4gI68=
-github.com/golang/glog v1.2.0/go.mod h1:6AhwSGph0fcJtXVM/PEHPqZlFeoLxhs7/t5UDAwmO+w=
+github.com/golang/glog v1.2.1 h1:OptwRhECazUx5ix5TTWC3EZhsZEHWcYWY4FQHTIubm4=
+github.com/golang/glog v1.2.1/go.mod h1:6AhwSGph0fcJtXVM/PEHPqZlFeoLxhs7/t5UDAwmO+w=
 github.com/golang/groupcache v0.0.0-20160516000752-02826c3e7903/go.mod h1:cIg4eruTrX1D+g88fzRXU5OdNfaM+9IcxsU14FzY7Hc=
 github.com/golang/groupcache v0.0.0-20190702054246-869f871628b6/go.mod h1:cIg4eruTrX1D+g88fzRXU5OdNfaM+9IcxsU14FzY7Hc=
 github.com/golang/groupcache v0.0.0-20191227052852-215e87163ea7/go.mod h1:cIg4eruTrX1D+g88fzRXU5OdNfaM+9IcxsU14FzY7Hc=