Skip to content

PLUGINAPI-180 New API for issue resolution#276

Merged
ericg138 merged 1 commit intomasterfrom
eg/sonar-resolve-api
Feb 25, 2026
Merged

PLUGINAPI-180 New API for issue resolution#276
ericg138 merged 1 commit intomasterfrom
eg/sonar-resolve-api

Conversation

@ericg138
Copy link
Contributor

For SonarSourcers:

Check before making the PR ready for review

  • Create a JIRA ticket if the API is impacted
  • Prefix the commit message with the ticket number
  • Document the change in CHANGELOG.md
  • When adding a new API:
    • Explain in the JavaDoc the purpose of the new API
    • Add a @since X.Y in the JavaDoc
  • When deprecating an API:
    • Annotate the deprecated element with @Deprecated
    • Add a @deprecated since X.Y in the JavaDoc
    • Document the replacement in the JavaDoc (if any)
  • When dropping an API:
  • Make sure the tests adhere to the convention:
    • All test method names should use snake_case, for example: test_validate_input. It can also start with the methodName
  • Make sure checks are green: build passes, Quality Gate is green
  • Merge after getting approval by at least one member of the guild
    • If no review is made within 3 days, gently ping the reviewers
    • The guild member reviewing the code can explicitly request someone else (typically another guild member representing another team) to check the impact on the specific product
    • In some cases, the guild may deem it necessary that multiple or even all members approve a PR. This is more likely in complex changes or changes directly impacting all teams using the API.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title New API for sonar-resolve code annotations PLUGINAPI-180 New API for sonar-resolve code annotations Feb 13, 2026
@hashicorp-vault-sonar-prod
Copy link

hashicorp-vault-sonar-prod bot commented Feb 13, 2026

PLUGINAPI-180

@ericg138 ericg138 force-pushed the eg/sonar-resolve-api branch 2 times, most recently from 36c93a5 to 875c62d Compare February 13, 2026 10:43
@ericg138 ericg138 force-pushed the eg/sonar-resolve-api branch from fa6e4ff to 89673ce Compare February 20, 2026 09:29
@ericg138 ericg138 marked this pull request as ready for review February 20, 2026 09:29
@ericg138 ericg138 requested a review from a team as a code owner February 20, 2026 09:29
@ericg138 ericg138 requested a review from henryju February 20, 2026 09:29
@ericg138 ericg138 changed the title PLUGINAPI-180 New API for sonar-resolve code annotations PLUGINAPI-180 New API for issue resolution Feb 20, 2026
Copy link
Member

@henryju henryju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor comment, else LGTM

@ericg138 ericg138 force-pushed the eg/sonar-resolve-api branch from 89673ce to b925741 Compare February 20, 2026 09:49
@ericg138 ericg138 force-pushed the eg/sonar-resolve-api branch from b925741 to aaecf71 Compare February 20, 2026 09:51
@sonarqube-next
Copy link

@ericg138
Copy link
Contributor Author

Hi @damien-urruty-sonarsource, just letting you know about this new API, it's probably something that will need to be implemented at some point in SQ for IDE.
Feel free to comment if you have any suggestions.

@damien-urruty-sonarsource
Copy link
Contributor

Thanks for the ping @ericg138. I will bring the topic to the squad, it's the first time I hear about that

@damien-urruty-sonarsource
Copy link
Contributor

@ericg138 I created https://sonarsource.atlassian.net/browse/SLCORE-2079.

As a first action we will keep the existing behavior and simply ignore those issues. PMs and UX will discuss a possible follow-up. Thanks!

@ericg138 ericg138 merged commit 558ded4 into master Feb 25, 2026
4 checks passed
@ericg138 ericg138 deleted the eg/sonar-resolve-api branch February 25, 2026 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants