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
Assume plugins that do not define a upper bound for Matomo version re… #15796
Conversation
Should we make a note in the CHANGELOG? And should we note this in the developer documentation? I wonder if it would be helpful (and even possible) to have an error message displaying the reason for the automatic < 4.0.0-b1? I think some developers may get confused, but I'm not too sure (cc @tsteur). |
Actually I guess we will need to update the marketplace and the developer docs. It seems we currently allow anything semver can parse. See https://developer.matomo.org/guides/distributing-your-plugin |
@sgiehl I've updated the marketplace for new plugin submissions or updates for existing plugins to require a lower and an upper limit. A developer will get an email with the error message and it will automatically suggest a good require string for that plugin when possible. Could you update the docs on developer site to have upper limits? |
…quirement as incompatible with Matomo 4
f85e9cb
to
df12761
Compare
Updated the PR and adjusted the tests. Would be ready for another review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Guess the only thing left is the developer docs tweak?
Merged the docs 👍 |
…quirement as incompatible with Matomo 4
fixes #14342