Skip to content
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

Remove integration with Static Analysis Utilities plugin #276

Merged
merged 1 commit into from
Feb 24, 2021

Conversation

basil
Copy link
Member

@basil basil commented Feb 16, 2021

See JENKINS-55744.

I first announced in the release notes for 2.78 that integration with the Static Analysis Utilities plugin was deprecated in the Email Extension plugin and would be removed in a future release. I posted the same deprecation warning in the corresponding Jira issue. I also updated the email template for this feature to include the same warning, mentioning the Jira issue. This last notification would be hard to miss, since all users who use this feature would begin seeing the warning in emails (regardless of whether or not they read Jenkins plugin release notes or Jira issues).

Furthermore, in the release notes for 2.81, I explicitly confirmed that integration with the Static Analysis Utilities plugin would be removed in the first quarter of 2021, regardless of whether or not anyone submits a pull request to add integration with the Warnings Next Generation plugin. I posted the same removal warning in the corresponding Jira issue.

It has now been about 4 months since this integration was deprecated, and I have received neither any pull requests to add integration with the Warnings Next Generation plugin nor any pleas from users to retain this deprecated feature. Therefore, as promised, I am going ahead with the removal of integration with the Static Analysis Utilities plugin.

@basil basil added the removed label Feb 16, 2021
@basil basil requested a review from uhafner February 16, 2021 16:37
@uhafner
Copy link
Member

uhafner commented Feb 17, 2021

Seems that no one is using emails nowadays. Since the analysis suite will be removed from the update center it makes sense to remove the template as well.

@basil basil merged commit 13bff82 into jenkinsci:master Feb 24, 2021
@basil basil deleted the JENKINS-55744 branch February 24, 2021 16:30
@timja
Copy link
Member

timja commented Mar 2, 2021

Could this be released, see jenkinsci/plugin-installation-manager-tool#294

(we will get a fix there too)

@basil
Copy link
Member Author

basil commented Mar 2, 2021

Thanks for pointing this out, @darxriggs and @timja. I will do a release of Email Extension sometime this week.

@basil
Copy link
Member Author

basil commented Mar 2, 2021

Released in 2.82.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants