Skip to content

Commit

Permalink
fix: minor comment cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
conorsch committed Nov 13, 2024
1 parent 2ebd1ae commit 5473b2d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion crates/bin/pd/src/migrate/mainnet1.rs
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ pub async fn migrate(
let start_time = std::time::SystemTime::now();

// Note, when this bit of code was added, the upgrade happened months ago,
// and the verison safeguard mechanism was not in place. However,
// and the version safeguard mechanism was not in place. However,
// adding this will prevent someone running version 0.80.X with the
// safeguard patch from accidentally running the migraton again, since they
// will already have version 8 written into the state. But, if someone is syncing
Expand Down
4 changes: 2 additions & 2 deletions crates/core/app/src/app_version/component.rs
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ async fn write_app_version_safeguard<S: StateWriteProto>(s: &mut S, x: u64) -> a
///
/// You should call this before starting a node.
///
/// This will succeed if no app version is saved, or if the app version saved matches
/// This will succeed if no app version was found in local storage, or if the app version saved matches
/// exactly.
///
/// This will also result in the current app version being stored, so that future
Expand All @@ -118,7 +118,7 @@ pub async fn assert_latest_app_version(s: Storage) -> anyhow::Result<()> {
///
/// This will check that the app version is currently the previous version, if set at all.
///
/// This is the only way to change the app version, and should be called during a migration
/// This is the recommended way to change the app version, and should be called during a migration
/// with breaking consensus logic.
pub async fn migrate_app_version<S: StateWriteProto>(s: &mut S, to: u64) -> anyhow::Result<()> {
anyhow::ensure!(to > 1, "you can't migrate to the first penumbra version!");
Expand Down

0 comments on commit 5473b2d

Please sign in to comment.