Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.Sign up
CLI exposes implementation detail "lamports" #8296
The CLI was originally written using all lamports, and then later, we added support for SOL. Now there's this fundamentally unnecessary option disambiguate between lamports and SOL, which is done by adding an argument with the text "lamports" or "SOL". That optional argument adds awkward edge cases, like, what happens if "SOL" is valid input for the next argument? Also, by retaining support for "lamports", we need to document it. Why brand 2 terms when 1 will suffice?
Remove support for "lamports" and remove support for specifying units.