refactor: Fix phpstan codeigniter.configArgumentInstanceof

[neznaika0]

Description
Modules does not extend BaseConfig. The object must be created directly by new Modules()

$modules instanceof Modules it is necessary, otherwise rector tries to overwrite __constructor(). For comparison with null, there are also rules - a change is acceptable here

Checklist:

• Securely signed commits
• Component(s) with PHPDoc blocks, only if necessary or adds value
• Unit testing, with >80% coverage
• User guide updated
• Conforms to style guide

[michalsn]

Shouldn't we allow Module config to be used with config()?

[neznaika0]

Modules do not extend BaseConfig and cannot have a Registrar. This is the same configuration as Autoloader, Paths. + they are not cached

[michalsn]

Yes, I know, but we're not talking about caching or registrars. This is a question, should the Module config have a single instance through Factories, as it is now? IMO yes.

[michalsn]

Seem like no one else is sharing my concern, so let's go.

[neznaika0]

Perhaps this is a problem for modular sites, who used config('Modules'). But this system files...