We just noticed that there are few plugins missing as submodules:
Can/should we add these as submodules maybe too to ensure we notice when we break these plugins? Not sure if it would cause tests to go a lot longer. Some of them also don't have tests.
Some of them are under the InnoCraft user. I suppose we could still add them? Or maybe otherwise move them to
matomo-org if that be an issue?
All plugins having tests should actually always run their tests against latest 4.x-dev branch. Imho it might be enough to run those tests regularly to ensure nothing breaks. Checking those builds before a release of Matomo might actually be enough.
@sgiehl are you saying we should remove all plugin submodules?
Not all. The only benefit I see on having the submodules is that we might be able to identify potential issues that might occur when plugin are available prior to installation or their code is updated together with core. See my last PRs in AnonymousTracking plugin. But those issues should be rare and unlikely and not worth the effort.
Btw Removing the submodules would mean updating a lot tests. But if we have set up a good process to check all plugins before we release a new version of Matomo we could probably do that.
For now it might be easier not to do that.