@tsteur opened this Issue on November 2nd 2021 Member

We just noticed that there are few plugins missing as submodules:

  • Migration
  • GoogleAnalyticsImporter
  • EnvironmentVariables
  • ForceSSL
  • Invalidate Reports
  • JS Tracker Custom
  • Custom Translations

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?

@sgiehl commented on November 2nd 2021 Member

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.

@tsteur commented on November 2nd 2021 Member

@sgiehl are you saying we should remove all plugin submodules?

@sgiehl commented on November 2nd 2021 Member

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.

Powered by GitHub Issue Mirror