New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add tracking spam prevention plugin as a submodule #17016
Conversation
@diosmosis @sgiehl I suppose it is normal that we run all the UI tests of a plugin with a submodule again? Like in https://builds-artifacts.matomo.org/matomo-org/matomo/4.x-dev/44659/ |
@tsteur I think it's because we consider the plugins bundled w/ core to be tested this often (testing compatibility on every matomo change). Otherwise the tests would only run when the plugin changes, even though a change in core could cause problems. |
Remaining test failures seem unrelated. Btw @tsteur did we decide if this plugin will be shipped with Matomo or if it will be available through Marketplace? We might need to adjust the build script maybe 🤔 |
@sgiehl it should be not included in core but instead it is already on the marketplace. Do we need to adjust anything? |
@sgiehl is this what we need to add re matomo-org/matomo#17016 (comment) ?
Description:
See https://github.com/matomo-org/plugin-TrackingSpamPrevention for the features that were added.
Review