Why are custom errors not de-duplicated across strategies?

[PaulRBerg]

Many custom errors, e.g. UNAUTHORIZED, are duplicated across multiple strategy contracts:

• allo-v2/contracts/strategies/direct-grants-simple/DirectGrantsSimpleStrategy.sol

  Line 48 in 04f84f8

  error UNAUTHORIZED();

• allo-v2/contracts/strategies/donation-voting/DonationVotingStrategy.sol

  Line 46 in 04f84f8

  error UNAUTHORIZED();

Why are they not de-duplicated by defining them only in BaseStrategy?

[PaulRBerg]

Alternatively, an errors library design could have been employed.

An example can be found in Sablier's V2 Core repository.

[codenamejason]

This is great feedback and honestly would love to put it on the todos.