Enhance the config commands and change some defaults.

.github/workflows/test.yml

@@ -53,7 +53,6 @@ jobs:
         run: |
           grep -vF \
             -e 'github.com/provenance-io/provenance/app' \
-            -e 'github.com/provenance-io/provenance/cmd/provenanced/cmd' \
             -e 'github.com/provenance-io/provenance/internal/antewrapper' \
             -e 'github.com/provenance-io/provenance/x/ibchooks' \
             -e 'github.com/provenance-io/provenance/x/ibcratelimit/module' \
@@ -63,7 +62,6 @@ jobs:
           split -d -n l/3 pkgs.txt.tmp pkgs.txt.part.
           printf '%s\n' \
             'github.com/provenance-io/provenance/app' \
-            'github.com/provenance-io/provenance/cmd/provenanced/cmd' \
             'github.com/provenance-io/provenance/internal/antewrapper' \
             'github.com/provenance-io/provenance/x/ibchooks' \
             'github.com/provenance-io/provenance/x/ibcratelimit/module' \

.golangci.yml

@@ -154,9 +154,6 @@ linters-settings:
           - github.com/cometbft/cometbft/types/time
           - github.com/cometbft/cometbft-db
 
-          - github.com/tendermint/tendermint
-          - github.com/tendermint/tm-db
-
           - github.com/rs/zerolog
       test:
         files:

CHANGELOG.md

@@ -72,6 +72,14 @@ Ref: https://keepachangelog.com/en/1.0.0/
 * Removes sync-info code for sdk v0.50 [#1760](https://github.com/provenance-io/provenance/issues/1760).
 * Fix most of the failing unit tests [#1943](https://github.com/provenance-io/provenance/pull/1943)
 * Clean up ReadFromClient [#1760](https://github.com/provenance-io/provenance/issues/1760).
+* Enhance the config get and changed commands to make it easier to find fields [#1968](https://github.com/provenance-io/provenance/pull/1968).
+* Change the default keyring backend to "os", but leave it as "test" for testnets [#1968](https://github.com/provenance-io/provenance/pull/1968).
+* Change the default broadcast mode to "sync" [#1968](https://github.com/provenance-io/provenance/pull/1968).
+* The pre-upgrade command now updates the client config's broadcast mode to "sync" if it's set to "block" [#1968](https://github.com/provenance-io/provenance/pull/1968).
+
+### Deprecated
+
+* In the config commands, the "tendermint" and "tm" options are deprecated, replaced with "cometbft", "comet", and "cmt" [#1968](https://github.com/provenance-io/provenance/pull/1968).
 
 ### Dependencies
 

CONTRIBUTING.md

@@ -63,9 +63,6 @@ not required to an open issue to submit a PR, but be aware that for more complex
 problems/features, if a PR is opened before an adequate design discussion has
 taken place in a github issue, that PR runs a high likelihood of being rejected.
 
-Take a peek at our [coding repo](https://github.com/tendermint/coding) for
-overall information on repository workflow and standards. Note, we use `make tools` for installing the linting tools.
-
 Other notes:
 
 - Looking for a good place to start contributing? How about checking out some

Makefile

@@ -14,7 +14,7 @@ endif
 
 BRANCH := $(shell git rev-parse --abbrev-ref HEAD 2> /dev/null)
 BRANCH_PRETTY := $(subst /,-,$(BRANCH))
-TM_VERSION := $(shell $(GO) list -m github.com/tendermint/tendermint 2> /dev/null | sed 's:.* ::') # grab everything after the space in "github.com/tendermint/tendermint v0.34.7"
+export CMTVERSION := $(shell $(GO) list -m github.com/cometbft/cometbft 2> /dev/null | sed 's:.* ::')
 COMMIT := $(shell git log -1 --format='%h' 2> /dev/null)
 # don't override user values
 ifeq (,$(VERSION))
@@ -108,7 +108,7 @@ ldflags = -w -s \
 	-X github.com/cosmos/cosmos-sdk/version.Version=$(VERSION) \
 	-X github.com/cosmos/cosmos-sdk/version.Commit=$(COMMIT) \
 	-X "github.com/cosmos/cosmos-sdk/version.BuildTags=$(build_tags_comma_sep)" \
-	-X github.com/tendermint/tendermint/version.TMCoreSemVer=$(TM_VERSION)
+	-X github.com/cometbft/cometbft/version.TMCoreSemVer=$(CMTVERSION)
 
 ldflags += $(LDFLAGS)
 ldflags := $(strip $(ldflags))

SECURITY.md

@@ -13,7 +13,7 @@ Please report security vulnerabilities to
 The Provenance team will send a response indicating the next steps in handling your
 report. After the initial reply to your report, the team will keep you informed
 of the progress towards remediation and may ask for additional
-information or guidance.  For critical problems you may encrypt your report using the public key below.
+information or guidance. For critical problems you may encrypt your report using the public key below.
 
 In addition, please include the following information along with your report:
 
@@ -27,7 +27,7 @@ an email to **[[email protected]](mailto:[email protected])**. The ema
 a short description of why it should be handled according to this security
 policy.
 
-If you have found an issue with the Cosmos SDK or Tendermint modules not found in this repo you can report them through links found here. https://tendermint.com/security/
+If you have found an issue with the Cosmos SDK or CometBFT modules not found in this repo you can report them through links found here: <https://github.com/cometbft/cometbft/blob/main/SECURITY.md>.
 
 ## Disclosure Policy
 

app/app.go

@@ -1296,7 +1296,7 @@ func (app *App) RegisterAPIRoutes(apiSvr *api.Server, apiConfig serverconfig.API
 	// Register new tx routes from grpc-gateway.
 	authtx.RegisterGRPCGatewayRoutes(clientCtx, apiSvr.GRPCGatewayRouter)
 
-	// Register new tendermint queries routes from grpc-gateway.
+	// Register new queries routes from grpc-gateway.
 	cmtservice.RegisterGRPCGatewayRoutes(clientCtx, apiSvr.GRPCGatewayRouter)
 
 	// Register node gRPC service for grpc-gateway.

app/test_helpers.go

@@ -45,8 +45,7 @@ import (
 	"github.com/provenance-io/provenance/internal/pioconfig"
 )
 
-// DefaultConsensusParams defines the default Tendermint consensus params used in
-// SimApp testing.
+// DefaultConsensusParams defines the default consensus params used in SimApp testing.
 var DefaultConsensusParams = &cmttmtypes.ConsensusParams{
 	Block: &cmttmtypes.BlockParams{
 		MaxBytes: 200000,