Tags: qspider/pocket-core
Tags
Improve version string comparison (pokt-network#1556) This patch changes the logic to compare the version strings to see if the network has been upgraded or not. The simple string comparison does not work when the network is upgraded from "0.9" to "0.10". The proposed fix is to convert a version string into a slice of integers before comparison. <!-- reviewpad:summarize:start --> ### Summary generated by Reviewpad on 08 Jun 23 00:08 UTC This pull request includes two patches. The first patch improves the version string comparison in `types/utils.go` by adding a new function called `CompareVersionStrings`. This function compares two version strings as if they are integers and returns negative, positive, or zero if the first string is earlier, later, or equal to the second string respectively. It also includes a new unit test for the function. The second patch addresses some of the previous comments and adds a new test case for the `CompareVersionStrings` function. <!-- reviewpad:summarize:end -->
Update tendermint version (pokt-network#1541) Update the tendermint fork version. ## Testing Validated use LocalNet instructions at [doc/guides/localnet.md](https://github.com/pokt-network/pocket-core/blob/staging/doc/guides/localnet.md) ## Changes - Update Tendermint to Go 1.18 + minor fixes: [tendermint/pull/74](pokt-network/tendermint#74) - Prevent peers who errored being added to the peer_set: [tendermint/pull/75](pokt-network/tendermint#75)
Addressing pokt-network#1501 by making set- and get-validator LeanPoc… …ket aware Several things are happening here under that title; for one, get-validator and set-validator now detect whether LeanPocket is on; if it is, then they output a message suggesting you might be trying to run the plural version of the commands and exits. The messages in set-validators and get-validators were similarly adapted to suggest you might be trying to run the singular version of the command before exiting. Last and least, a the setValidatorsLean command was slightly edited so the whitespace around the check for LeanPocket matches the others.
PreviousNext