chore: fix every typo that ever exists on the repo (osmosis-labs#7059)

* WIP * Generated protofile changes * proto * x * Generated protofile changes * no more fix typo * revert this * revert this as well * revert all changes to contracts * testing CI * chore: There are a few words left (osmosis-labs#7067) * leftover --------- Co-authored-by: github-actions <[email protected]> Co-authored-by: Dong Lieu <[email protected]>
likesToEatFish · Dec 12, 2023 · c78426a · c78426a
1 parent 92c9bfc
commit c78426a
Show file tree

Hide file tree

Showing 214 changed files with 456 additions and 454 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -87,7 +87,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 * [#6840](https://github.com/osmosis-labs/osmosis/pull/6840) fix: change TypeMsgUnbondConvertAndStake value to "unbond_convert_and_stake" and improve error message when epoch currentEpochStartHeight less than zero
 * [#6769](https://github.com/osmosis-labs/osmosis/pull/6769) fix: improve dust handling in EstimateTradeBasedOnPriceImpact
-* [#6841](https://github.com/osmosis-labs/osmosis/pull/6841) fix: fix receive_ack response field and imporove error message of InvalidCrosschainSwapsContract and NoDenomTrace
+* [#6841](https://github.com/osmosis-labs/osmosis/pull/6841) fix: fix receive_ack response field and improve error message of InvalidCrosschainSwapsContract and NoDenomTrace
 
 ## v20.4.0
 
@@ -209,7 +209,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * [#6238] switch osmomath to sdkmath types and rename BigDec constructors to contain "Big" in the name.
 
 Note: with the update, the Dec and Int do not get initialized to zero values by default in proto marhaling/unmarshaling. Instead, they get set to nil values.
-maxDecBitLen has changed by one bit so overflow panic can be triggerred sooner.
+maxDecBitLen has changed by one bit so overflow panic can be triggered sooner.
 
 ## v19.0.0
 
@@ -220,7 +220,7 @@ maxDecBitLen has changed by one bit so overflow panic can be triggerred sooner.
 ### Bug Fixes
 * [#6190](https://github.com/osmosis-labs/osmosis/pull/6190) v19 upgrade handler superfluid fix
 * [#6195](https://github.com/osmosis-labs/osmosis/pull/6195) (x/tokenfactory) Fix events for `mintTo` and `burnFrom`
-* [#6195](https://github.com/osmosis-labs/osmosis/pull/6195) Fix panic edge case in superfluid AfterEpochEnd hook by surrounding CL multipler update with ApplyFuncIfNoError
+* [#6195](https://github.com/osmosis-labs/osmosis/pull/6195) Fix panic edge case in superfluid AfterEpochEnd hook by surrounding CL multiplier update with ApplyFuncIfNoError
 
 ### Misc Improvements
 
@@ -256,7 +256,7 @@ Fixes mainnet bugs w/ incorrect accumulation sumtrees, and CL handling for a bal
 * [#6238](https://github.com/osmosis-labs/osmosis/pull/6238) switch osmomath to sdkmath types and rename BigDec constructors to contain "Big" in the name.
    * Note: with the update, the Dec and Int do not get initialized to zero values
    by default in proto marhaling/unmarshaling. Instead, they get set to nil values.
-   * maxDecBitLen has changed by one bit so overflow panic can be triggerred sooner.
+   * maxDecBitLen has changed by one bit so overflow panic can be triggered sooner.
 * [#6071](https://github.com/osmosis-labs/osmosis/pull/6071) reduce number of returns for UpdatePosition and TicksToSqrtPrice functions
 * [#5906](https://github.com/osmosis-labs/osmosis/pull/5906) Add `AccountLockedCoins` query in lockup module to stargate whitelist.
 * [#6053](https://github.com/osmosis-labs/osmosis/pull/6053) monotonic sqrt with 36 decimals
@@ -300,7 +300,7 @@ Fixes mainnet bugs w/ incorrect accumulation sumtrees, and CL handling for a bal
 ### Misc Improvements
 
 * [#5534](https://github.com/osmosis-labs/osmosis/pull/5534) fix: fix the account number of x/tokenfactory module account
-* [#5750](https://github.com/osmosis-labs/osmosis/pull/5750) feat: add cli commmand for converting proto structs to proto marshalled bytes
+* [#5750](https://github.com/osmosis-labs/osmosis/pull/5750) feat: add cli command for converting proto structs to proto marshalled bytes
 * [#5889](https://github.com/osmosis-labs/osmosis/pull/5889) provides an API for protorev to determine max amountIn that can be swapped based on max ticks willing to be traversed
 * [#5849](https://github.com/osmosis-labs/osmosis/pull/5849) CL: Lower gas for leaving a position and withdrawing rewards
 * [#5855](https://github.com/osmosis-labs/osmosis/pull/5855) feat(x/cosmwasmpool): Sending token_in_max_amount to the contract before running contract msg
@@ -475,7 +475,7 @@ Same changes included in `v15.1.2` but redacted as tagged commit was not part of
 
 ## v15.0.0
 
-This release containts the following new modules:
+This release contains the following new modules:
 - ProtoRev module (x/protorev). This module captures MEV via in-protocol cyclic arbitrage and distributes the revenue back to the protocol based on governance. Developed by the Skip team.
 - Validator Set Preference module (x/valset-pref). This module gives users the ability to delegate to multiple validators according to their preference list.
 - Pool Manager module (x/poolmanager). This module manages the infrastructure around pool creation and swaps. It serves as a unified entrypoint for any swap related message or query. This module is extracted from the pre-existing `x/gamm`. It is the first milestone on the path towards delivering concentrated liquidity.
@@ -637,7 +637,7 @@ This release includes several cosmwasm-developer and appchain-ecosystem affectin
 * TWAP - Time weighted average prices for all AMM pools
 * Cosmwasm contract developer facing features
   * Enabling select queries for cosmwasm contracts
-  * Add message responses to gamm messages, to remove the neccessity of bindings
+  * Add message responses to gamm messages, to remove the necessity of bindings
   * Allow specifying denom metadata from tokenfactory
 * Enabling Interchain accounts (for real this time)
 * Upgrading IBC to v3.3.0
@@ -834,7 +834,7 @@ Cosmwasm bindings have been added, to make swapping and creating these new token
 * [#1435](https://github.com/osmosis-labs/osmosis/pull/1435) `x/tokenfactory` create denom fee for spam resistance 
 * [#1253](https://github.com/osmosis-labs/osmosis/pull/1253) Add a message to increase the duration of a bonded lock.
 * [#1656](https://github.com/osmosis-labs/osmosis/pull/1656) Change camelCase to snake_case in proto.
-* [#1632](https://github.com/osmosis-labs/osmosis/pull/1632) augment SuperfluidDelegationsByDelegator query, return osmo equivilent is staked via superfluid
+* [#1632](https://github.com/osmosis-labs/osmosis/pull/1632) augment SuperfluidDelegationsByDelegator query, return osmo equivalent is staked via superfluid
 * [#1723](https://github.com/osmosis-labs/osmosis/pull/1723) fix number of LP shares returned from stableswap pool
 
 ## [v8.0.0 - Emergency proposals upgrade](https://github.com/osmosis-labs/osmosis/releases/tag/v8.0.0)
@@ -881,7 +881,7 @@ Every node should upgrade their software version to v8.0.0 before the upgrade bl
 
 ### Minor improvements & Bug Fixes
 
-* [#1442](https://github.com/osmosis-labs/osmosis/pull/1442) Use latest tm-db release for badgerdb and rocksdb improvments
+* [#1442](https://github.com/osmosis-labs/osmosis/pull/1442) Use latest tm-db release for badgerdb and rocksdb improvements
 * [#1379](https://github.com/osmosis-labs/osmosis/pull/1379) Introduce `Upgrade` and `Fork` structs, to simplify upgrade logic.
 * [#1363](https://github.com/osmosis-labs/osmosis/pull/1363) Switch e2e test setup to create genesis and configs via Dockertest
 * [#1335](https://github.com/osmosis-labs/osmosis/pull/1335) Add utility for deriving total orderings from partial orderings.
@@ -892,7 +892,7 @@ Every node should upgrade their software version to v8.0.0 before the upgrade bl
 * [#1193](https://github.com/osmosis-labs/osmosis/pull/1193) Setup e2e tests on a single chain; add balances query test
 * [#1095](https://github.com/osmosis-labs/osmosis/pull/1095) Fix authz being unable to use lockup & superfluid types.
 * [#1105](https://github.com/osmosis-labs/osmosis/pull/1105) Add GitHub Actions to automatically push the osmosis Docker image
-* [#1114](https://github.com/osmosis-labs/osmosis/pull/1114) Improve CI: remove duplicate runs of test worflow
+* [#1114](https://github.com/osmosis-labs/osmosis/pull/1114) Improve CI: remove duplicate runs of test workflow
 * [#1127](https://github.com/osmosis-labs/osmosis/pull/1127) Stricter Linting:  bump golangci-lint version and enable additional linters.
 * [#1184](https://github.com/osmosis-labs/osmosis/pull/1184) Fix endtime event output on BeginUnlocking
 
@@ -996,7 +996,7 @@ The v7 release introduces Superfluid Staking! This allows governance-approved LP
 * [#763](https://github.com/osmosis-labs/osmosis/pull/763) Add rocksDB options to Makefile.
 * [#740](https://github.com/osmosis-labs/osmosis/pull/740) Simplify AMM swap math / file structure.
 * [#731](https://github.com/osmosis-labs/osmosis/pull/731) Add UpdateFeeToken proposal handler to app.go
-* [#686](https://github.com/osmosis-labs/osmosis/pull/686) Add silence usage to cli to surpress unnecessary help logs
+* [#686](https://github.com/osmosis-labs/osmosis/pull/686) Add silence usage to cli to suppress unnecessary help logs
 * [#652](https://github.com/osmosis-labs/osmosis/pull/652) Add logic for deleting a pool
 * [#541](https://github.com/osmosis-labs/osmosis/pull/541) Start generalizing the AMM infrastructure
 
@@ -1075,7 +1075,7 @@ The v7 release introduces Superfluid Staking! This allows governance-approved LP
 * Update to Tendermint v0.34.15
 * Increase p2p timeouts to alleviate p2p network breaking at epoch
 * [#741](https://github.com/osmosis-labs/osmosis/pull/741) Allow node operators to set a second min gas price for arbitrage txs.
-* [#623](https://github.com/osmosis-labs/osmosis/pull/623) Use gosec for staticly linting for common non-determinism issues in SDK applications.
+* [#623](https://github.com/osmosis-labs/osmosis/pull/623) Use gosec for statically linting for common non-determinism issues in SDK applications.
 
 ## Minor improvements & Bug Fixes
 
@@ -1136,7 +1136,7 @@ Upgrade instructions for node operators can be found [here](https://github.com/o
 * {In the Osmosis-labs SDK fork}
   * Increase default IAVL cache size to be in the hundred megabyte range
   * Significantly improve CacheKVStore speed problems, reduced IBC upgrade time from 2hrs to 5min
-  * Add debug info to make it clear whats happening during upgrade
+  * Add debug info to make it clear what's happening during upgrade
 * (From a series of commits) Fixes to the claims module to only do the reclaim logic once, not every block.
 * (From a series of commits) More logging to the claims module.
 * [\#563](https://github.com/osmosis-labs/osmosis/pull/563) Allow zero-weight pool-incentive distribution records
@@ -1168,7 +1168,7 @@ Upgrade instructions for node operators can be found [here](https://github.com/o
 
 * Update the cosmos-sdk version we modify to v0.42.9
 * Fix a bug in the min commission rate code that allows validators to be created with commission rates less than the minimum.
-* Automatically upgrade any validator with less than the minimum comission rate to the minimum at upgrade time.
+* Automatically upgrade any validator with less than the minimum commission rate to the minimum at upgrade time.
 * Unbrick on-chain governance, by fixing the deposit parameter to use `uosmo` instead of `osmo`.
 
 ## [v1.0.2](https://github.com/osmosis-labs/osmosis/releases/tag/v1.0.2) - 2021-06-18

diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -333,7 +333,7 @@ For v6.x, and v4.x, most PRs to them should go to main and get a "backport" labe
 
 ### How to build proto files. (rm -rf vendor/ && make build-reproducible once docker is installed)
 
-You can do rm -rf vendor and make build-reproducible to redownload all dependencies - this should pull the latest docker image of Osmosis. You should also make sure to do make proto-all to auto-generate your protobuf files. Makes ure you have docker installed.
+You can do rm -rf vendor and make build-reproducible to redownload all dependencies - this should pull the latest docker image of Osmosis. You should also make sure to do make proto-all to auto-generate your protobuf files. Makes sure you have docker installed.
 
 If you get something like `W0503 22:16:30.068560 158 services.go:38] No HttpRule found for method: Msg.CreateBalancerPool` feel free to ignore that.
 

diff --git a/Makefile b/Makefile
@@ -191,7 +191,7 @@ sqs-profile:
 sqs-validate-cl-state:
 	ingest/sqs/scripts/validate-cl-state.sh "http://localhost:9092"
 
-# Compares the quotes betwen SQS and chain over pool 1136
+# Compares the quotes between SQS and chain over pool 1136
 # which is concentrated.
 sqs-quote-compare:
 	ingest/sqs/scripts/quote.sh "http://localhost:9092"

diff --git a/app/app.go b/app/app.go
@@ -298,7 +298,7 @@ func NewOsmosisApp(
 	// Generally NewAppModule will require the keeper that module defines to be passed in as an exact struct,
 	// but should take in every other keeper as long as it matches a certain interface. (So no need to be de-ref'd)
 	//
-	// Any time a module requires a keeper de-ref'd thats not its native one,
+	// Any time a module requires a keeper de-ref'd that's not its native one,
 	// its code-smell and should probably change. We should get the staking keeper dependencies fixed.
 	app.mm = module.NewManager(appModules(app, encodingConfig, skipGenesisInvariants)...)
 

diff --git a/app/apptesting/test_suite.go b/app/apptesting/test_suite.go
@@ -50,7 +50,7 @@ type KeeperTestHelper struct {
 
 	// defaults to false,
 	// set to true if any method that potentially alters baseapp/abci is used.
-	// this controls whether or not we can re-use the app instance, or have to set a new one.
+	// this controls whether or not we can reuse the app instance, or have to set a new one.
 	hasUsedAbci bool
 	// defaults to false, set to true if we want to use a new app instance with caching enabled.
 	// then on new setup test call, we just drop the current cache.
@@ -183,7 +183,7 @@ func (s *KeeperTestHelper) PrepareAllSupportedPools() SupportedPoolAndGaugeInfo
 // On first reset, will instantiate a new app, with caching enabled.
 // NOTE: If you are using ABCI methods, usage of Reset vs Setup has not been well tested.
 // It is believed to work, but if you get an odd error, try changing the call to this for setup to sanity check.
-// whats supposed to happen is a new setup call, and reset just does that in such a case.
+// what's supposed to happen is a new setup call, and reset just does that in such a case.
 func (s *KeeperTestHelper) Reset() {
 	if s.hasUsedAbci || !s.withCaching {
 		s.withCaching = true

diff --git a/app/keepers/keepers.go b/app/keepers/keepers.go
@@ -557,7 +557,7 @@ func (appKeepers *AppKeepers) InitNormalKeepers(
 // * RecvPacket, message that originates from core IBC and goes down to app, the flow is the other way
 // channel.RecvPacket -> ibc_hooks.OnRecvPacket -> ibc_rate_limit.OnRecvPacket -> forward.OnRecvPacket -> transfer.OnRecvPacket
 //
-// Note that the forward middleware is only integrated on the "reveive" direction. It can be safely skipped when sending.
+// Note that the forward middleware is only integrated on the "receive" direction. It can be safely skipped when sending.
 // Note also that the forward middleware is called "router", but we are using the name "forward" for clarity
 // This may later be renamed upstream: https://github.com/ibc-apps/middleware/packet-forward-middleware/issues/10
 //

diff --git a/app/modules.go b/app/modules.go
@@ -205,7 +205,7 @@ func orderBeginBlockers(allModuleNames []string) []string {
 	// TODO: Perhaps this can be relaxed, left to future work to analyze.
 	ord.Sequence(distrtypes.ModuleName, slashingtypes.ModuleName, evidencetypes.ModuleName, stakingtypes.ModuleName)
 	// superfluid must come after distribution & epochs.
-	// TODO: we actually set it to come after staking, since thats what happened before, and want to minimize chance of break.
+	// TODO: we actually set it to come after staking, since that's what happened before, and want to minimize chance of break.
 	ord.After(superfluidtypes.ModuleName, stakingtypes.ModuleName)
 	// TODO: This can almost certainly be un-constrained, but we keep the constraint to match prior functionality.
 	// IBChost came after staking, before superfluid.
@@ -231,7 +231,7 @@ func OrderEndBlockers(allModuleNames []string) []string {
 
 // OrderInitGenesis returns module names in order for init genesis calls.
 func OrderInitGenesis(allModuleNames []string) []string {
-	// NOTE: The genutils moodule must occur after staking so that pools are
+	// NOTE: The genutils module must occur after staking so that pools are
 	// properly initialized with tokens from genesis accounts.
 	// NOTE: Capability module must occur first so that it can initialize any capabilities
 	// so that other modules that want to create or claim capabilities afterwards in InitChain

diff --git a/app/params/doc.go b/app/params/doc.go
@@ -3,9 +3,9 @@ Package params defines the simulation parameters in the simapp.
 
 It contains the default weights used for each transaction used on the module's
 simulation. These weights define the chance for a transaction to be simulated at
-any gived operation.
+any given operation.
 
-You can repace the default values for the weights by providing a params.json
+You can replace the default values for the weights by providing a params.json
 file with the weights defined for each of the transaction operations:
 
 	{

diff --git a/app/upgrades/types.go b/app/upgrades/types.go
@@ -10,7 +10,7 @@ import (
 	"github.com/osmosis-labs/osmosis/v21/app/keepers"
 )
 
-// BaseAppParamManager defines an interrace that BaseApp is expected to fullfil
+// BaseAppParamManager defines an interrace that BaseApp is expected to fulfill
 // that allows upgrade handlers to modify BaseApp parameters.
 type BaseAppParamManager interface {
 	GetConsensusParams(ctx sdk.Context) *tmproto.ConsensusParams

diff --git a/app/upgrades/v13/upgrade_test.go b/app/upgrades/v13/upgrade_test.go
@@ -60,7 +60,7 @@ func (s *UpgradeTestSuite) TestUpgrade() {
 				// acc := s.App.AccountKeeper.GetAccount(s.Ctx, ibc_hooks.WasmHookModuleAccountAddr)
 				// s.App.AccountKeeper.RemoveAccount(s.Ctx, acc)
 
-				// Because of SDK version map bug, we can't do the following, and instaed do a massive hack
+				// Because of SDK version map bug, we can't do the following, and instead do a massive hack
 				// vm := s.App.UpgradeKeeper.GetModuleVersionMap(s.Ctx)
 				// delete(vm, ibchookstypes.ModuleName)
 				// OR

diff --git a/app/upgrades/v17/upgrades_test.go b/app/upgrades/v17/upgrades_test.go
@@ -57,7 +57,7 @@ func dummyUpgrade(s *UpgradeTestSuite) {
 	s.Ctx = s.Ctx.WithBlockHeight(dummyUpgradeHeight)
 }
 
-func dummyTwapRecord(poolId uint64, t time.Time, asset0 string, asset1 string, sp0, accum0, accum1, geomAccum osmomath.Dec) types.TwapRecord { //nolint:unparam // asset1 aleays receives "usomo"
+func dummyTwapRecord(poolId uint64, t time.Time, asset0 string, asset1 string, sp0, accum0, accum1, geomAccum osmomath.Dec) types.TwapRecord { //nolint:unparam // asset1 always receives "usomo"
 	return types.TwapRecord{
 		PoolId:      poolId,
 		Time:        t,

diff --git a/app/upgrades/v18/upgrades_test.go b/app/upgrades/v18/upgrades_test.go
@@ -80,7 +80,7 @@ func (s *UpgradeTestSuite) TestUpgrade() {
 	s.App.BeginBlocker(s.Ctx, abci.RequestBeginBlock{})
 	s.Ctx = s.Ctx.WithBlockTime(s.Ctx.BlockTime().Add(time.Hour * 24))
 
-	// after the accum values have been resetted correctly after upgrade, we expect the accumulator store to be initialized with the correct value,
+	// after the accum values have been reset correctly after upgrade, we expect the accumulator store to be initialized with the correct value,
 	// which in our test case would be 10000(the amount that was locked)
 	valueAfterClear := s.App.LockupKeeper.GetPeriodLocksAccumulation(s.Ctx, lockuptypes.QueryCondition{
 		LockQueryType: lockuptypes.ByDuration,
@@ -186,7 +186,7 @@ func (s *UpgradeTestSuite) setupCorruptedState() {
 	s.Require().NoError(err)
 
 	// also create a lock with the shares that would stay locked during the upgrade.
-	// doing this would help us assert if the accumulator has been resetted to the correct value.
+	// doing this would help us assert if the accumulator has been reset to the correct value.
 	shareCoinsStaysLocked := sdk.NewCoins(sdk.NewCoin(aktSharesDenom, osmomath.NewInt(shareStaysLocked)))
 	s.FundAcc(addr, shareCoinsStaysLocked)
 	_, err = keepers.LockupKeeper.CreateLock(s.Ctx, addr, shareCoinsStaysLocked, time.Hour*24*14)
@@ -219,7 +219,7 @@ func (s *UpgradeTestSuite) setupCorruptedState() {
 }
 
 // We want to ensure that with the corrupted state of the lockup accumulator,
-// `AfterEpochEnd` was panicing.
+// `AfterEpochEnd` was panicking.
 // We can do this check by creating a CL pool, then trying to distribute using that specific
 // CL pool gauge. If our test setup was correct, this should panic.
 func (s *UpgradeTestSuite) ensurePreUpgradeDistributionPanics() {