Remove unnecessary todo

client/v2/internal/coins/format.go

@@ -11,7 +11,6 @@ import (
 // Amount can be a whole number or a decimal number. Denominations can be 3 ~ 128
 // characters long and support letters, followed by either a letter, a number or
 // a separator ('/', ':', '.', '_' or '-').
-// TODO: Update?
 var coinRegex = regexp.MustCompile(`^(\d+(\.\d+)?)([a-zA-Z][a-zA-Z0-9\/\:\._\-]{2,127})$`)
 
 // ParseCoin parses a coin from a string. The string must be in the format

types/coin.go

@@ -875,7 +875,7 @@ func ValidateDenom(denom string) error {
 }
 
 // isValidRune checks if a given rune is a valid character for a rune.
-// It returns true if the rune is a letter, digit, '/', ':', '.', '_', or '-'.
+// It returns true if the rune is a letter, digit, '/', ':', '.', '_', '-', or '$'.
 func isValidRune(r rune) bool {
 	return unicode.IsLetter(r) || unicode.IsDigit(r) || r == '/' || r == ':' || r == '.' || r == '_' || r == '-' || r == '$'
 }