I’m trying to update my plugin for 8.1 and I’ve noticed a couple of confusing changes about the plugin api artifact, which I don’t think were communicated anywhere in the docs, so I’m writing here hoping to get some clarification.
The first thing I’ve noticed is the versioning scheme in 8.1, which looks different from all the other versions published previously. There seem to be two releases of the 8.1 plugin api artifact, both published on the same day:
Was that intentional or just simply a mistake? If the former, does that mean a change in the versioning scheme used going forward? This format does not appear to be conformant with semantic versioning (the
+ character should be used to denote build metadata, e.g.
8.1.0+31237) - is that something you’re aware of and happy with? Is SonarQube server going to be able to parse a version defined by plugins in this format and determine correctly plugin compatibility given format isn’t a valid semver scheme?
What are the implications for plugin developers? I’m specifically referring to the
Sonar-Version from the manifest, which defines the minimum version of supported SonarQube runtime. I’m assuming the earlier version needs to be targeted by plugin developers wanting to support 8.1, or was it released accidentally and SonarQube 184.108.40.206197 was never available for download?
The second thing I wanted to ask about is the new sonar-plugin-api-impl artifact, which seems to be necessary to be included as a test dependency in my plugin to get it compiled against 8.1. It took me a little bit of digging through the history on Github to figure out that I needed to include it since it contains all the
internal packages which I’m making use of for testing purposes, such as the following (just to name a few):
Is that still the correct use to include this new module as a test dependency or do you have a replacement for the functionality provided by those internal packages somewhere else? It would be good to get this change documented for others as I was a little bit shocked by seeing ~250 compiler errors before discovering the internal packages were extracted to a separate module. It’s probably something that should be included either in the release notes, in a dedicated section for developers, or in the plugin API changes part of the docs.