Fixed Coding Standards recommendations

MR with the fix: https://git.drupalcode.org/project/hook_event/-/merge_requests/3

I agree on the points for README and mixed typehint. I did found a couple of other phpcs issues pending related to DI.

FILE: src/Discovery/HookEventsDiscovery.php
---------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
---------------------------------------------------------------------------------------------
 64 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 66 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
---------------------------------------------------------------------------------------------

I was thinking about fix on this current issue focus on "Coding Standards", and have a separated ticket to handle the "Best Practice" where we can fix the DI cases cited

In that case, this can be marked RTBC.

Thx, RenatoG! For the mixed type, it's as you said - the PHP 8.0+ feature, so it should stay as it is.
I also understand the point of ignoring the README.md, having code samples makes it almost impossible to keep with that limit (I tend to like the PSR-12 120 chars limit BTW).

Regarding the DI, I really wanted to push this module to the stable release.
This was the only piece of the puzzle that I couldn't figure out. I was trying to make it
the same way as the circular reference is done at the core for the theme.manager

  theme.manager:
    class: Drupal\Core\Theme\ThemeManager
    arguments: ['%app.root%', '@theme.negotiator', '@theme.initialization', '@module_handler']
    calls:
      - [setThemeRegistry, ['@theme.registry']]

strangely without any luck.
I don't have that much time recently so any help on this issue will be appreciated.

Automatically closed - issue fixed for 2 weeks with no activity.