fix: remove unnessecary move from convert method

[affanshahid]

ExchangeRate::convert moves the amount parameter even though it doesn't need to. Changed this to an immutable reference to prevent unnecessary moves/clones in user code.

[varunsrin]

good catch, thanks!