refacto: clean indentation of the smart contracts

[opixelum]

I used Prettier for formatting all the contracts with 4 spaces for indentation, and I also used the option "preserve" for "explicitTypes" to respect the type used by the developers.

[n1c01a5]

There is an indentation issue error.

[n1c01a5]

And use the prettier from OZ: see https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/.prettierrc ; to follow the reference.

[opixelum]

Indentation fixed. Then, do you prefer to always use explicit types, as OZ's Prettier config does, or I set this option to "preserve" ? Because before editing the contracts, I saw that you were not using explicit types.

[n1c01a5]

I prefer to use the same set up than OZ so with explicit types preserve.

In general, it's better to follow the reference.

[n1c01a5]

@opixelum did you run the tests to see if everything works ?

[opixelum]

@n1c01a5 Yes, all tests passed ✅.

However, we have some warnings about shadowing existing declarations and deprecated methods (those were already here before the formatting).

I can see how to remove those warnings if needed.

[n1c01a5]

@n1c01a5 Yes, all tests passed ✅.

However, we have some warnings about shadowing existing declarations and deprecated methods (those were already here before the formatting).

I can see how to remove those warnings if needed.

Perfect ! These warnings are not really relevant 👌