* Update ChainGovernorResetReleaseTimerRequest protobuf message
* Add numDays argument to governor-reset-release-timer command
* Update governor backend to support numDays argument
* Address review comments
* Add test for resetReleaseTimerForTime()'s numDays parameter
* Address review comments
* Add adminrpc test for ChainGovernorResetReleaseTimer
* Replace hardcoded upper boundaries with maxResetReleaseTimerDays
* Update governor whitepaper to reflect the new argument
* Added default value to governor whitepaper
---------
Co-authored-by: Jason Matthyser <jason@asymmetric.re>
* Add cspell configuration and custom dictionary
The goal is to cut down on both incoming tyops, and well meaning but
spammy tyop fix PRs.
To run cspell locally install it and run:
cspell '**/*.md' \
--config cspell.config.yaml \
--words-only \
--unique \
--quiet | sort --ignore-case
* docs: cspell updates
* wormchain: cspell updates
* aptos: cspell updates
* node: cspell updates
* algorand: cspell updates
* whitepapers: cspell updates
* near: cspell updates
* solana: cspell updates
* terra: cspell updates
* cosmwasm: cspell updates
* ethereum: cspell updates
* clients: cspell updates
* cspell updates for DEVELOP document
* github: run cspell github action
* sdk: cspell updates
* github: only run cspell on markdown files
* algorand: EMMITTER --> EMITTER
Suggested-by: @evan-gray
* cspell: removed from dictionary
Suggested-by: @evan-gray
* aptos and node: cspell updates
Suggested-by: @evan-gray
* cosmowasm: doc updates for terra2
Suggested-by: @evan-gray
* algorand: cspell updates
Suggested-by: @evan-gray
* algorand: cspell updates
Suggested-by: @evan-gray
* cspell: updated custom word dictionary
This resorts the dictionary and adds a few new words from the
algorand/MEMORY.md document around varints and integers.
* cspell: sort the dictionary how vscode does it
On macOS the sorting is locale dependent. To do this on macOS, you have
to invert the case, do a character insensitive sort, and then invert the
case again:
LC_COLLATE="en_US.UTF-8" cspell '**/*.md' --config cspell.config.yaml \
--words-only \
--unique \
--no-progress \
--quiet \
| tr 'a-zA-Z' 'A-Za-z' \
| sort --ignore-case \
| tr 'a-zA-Z' 'A-Za-z'
This requires the `LC_COLLATE` variable to be set to `en_US.UTF-8`, or it
will not do the right thing.
* docs: grammar clean up
---------
Co-authored-by: Evan Gray <battledingo@gmail.com>
* Node/CCQ/Solana: Add sol_pda query
* Attempting to fix bigint serialize error in tests
* Try backing out sol_pda tests
* Put some of solana.test.ts changes back
* Add more stuff back
* Add more stuff to solana.test.ts
* Add more solana.test.ts stuff
* Whatever
* More sol_pda test debugging
* Code review rework
* More rework
* whitepapers: First version of integrity checkers and global accountant
* whitepapers: revise accountant whitepaper
* whitepapers: updates for accountant papers
* Wormhole is connected to both L1 and L2 networks, so L1 was replaced
with blockchain or network where appropriate.
* Change integrity checker "advantages" to "disadvantages" as that is
what is trying to be explained
* Explain that a pre-observation is just a concept a bit more as
the actual global-accountant contract function name is
`send_observations()` not `submit_preobservations()`
* Removed a lot of passive voice and tries to shorten run-on sentences
* Removed the integrity checkers Todo and Caveats as they are empty
* Moved the Integrity Checkers `Detailed Design` section to just a
terminology section, which is more appropriate for what it was.
* whitepapers: accountant clarifications
Addressing review feedback from: @conorpp and @hendrikhofstadt
This updates the Submitting Observations to Wormchain section to use
`transfer` in place of a mix of transfer and observation. This makes the
text easier to understand. Additionally, it moves the `Pending` status
line up one bullet point to make it more obvious.
* whitepapers: accountant further clarification
Last review comment from @conorpp
* whitepapers: update accountant nits
* whitepapers: addreses review feedback
Thanks @bruce-riley
---------
Co-authored-by: tbjump <unknown>
Co-authored-by: Conor Patrick <conorpp94@gmail.com>
Co-authored-by: Jeff Schroeder <jeffschroeder@computer.org>
Opening this PR for open feedback on a security feature that Guardians can optionally opt into that would limit the amount of notional value that can be transferred out of a given chain in a given time frame -- this would reduce the damage to the bridge during an existential threat scenario.