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

Refresh plugin for August 2023 (Upgrade to a minimum core version 2.361.3 and minimum Java version 11) #348

Merged
merged 3 commits into from
Sep 21, 2023

Conversation

basil
Copy link
Member

@basil basil commented Aug 29, 2023

No description provided.

@basil
Copy link
Member Author

basil commented Aug 29, 2023

Currently blocking jenkinsci/bom#2440

@basil basil requested a review from a team August 29, 2023 23:02
[platform: 'windows', jdk: 8],
buildPlugin(useContainerAgent: true, configurations: [
[platform: 'linux', jdk: 17],
[platform: 'windows', jdk: 11],
Copy link
Member

Choose a reason for hiding this comment

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

if no more java 8 support maybe worth to add the minimum jdk requirement in the PR title as is this will be in the release notes

Copy link
Member Author

Choose a reason for hiding this comment

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

Am I understanding correctly that you are a maintainer of this repository and can therefore edit PR titles?

Copy link
Member

Choose a reason for hiding this comment

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

I''m really sorry for the delay and misunderstanding. But as you are the owner of the current work (this PR) I was under the impression you would have been the most appropriate to modify your own work after review from the member of this project's community. I'm not usually modifying work from others.
But so as it looks to take time, I will do it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks. I generally don't include minimum core version changes in release notes for the plugins I maintain (unless the minimum core version is unusual, like a weekly release or a very recent LTS) because such plugin releases aren't even advertised in the UI to users of older cores. Similarly, a recent core version implies a minimum Java version, and users running older Java versions (and therefore older cores) wouldn't get an advertisement for such a plugin upgrade in the UI, so I generally don't include minimum Java version changes in release notes. I have filed dozens (hundreds?) of PRs without this information in the title and shipped dozens of releases without this information in the release notes, all without any complaints from users or other developers. But I acknowledge that as the plugin maintainer, you should be free to include (or not include) any information you want in the release notes. I hope that this discussion about writing release notes has not been a significant distraction from the comparatively far more laborious task of modernizing the plugin itself.

@basil
Copy link
Member Author

basil commented Sep 7, 2023

Currently blocking jenkinsci/bom#2440

@olamy olamy changed the title Refresh plugin for August 2023 Refresh plugin for August 2023 (Upgrade to a minimum core version 2.361.3 and minimum Java version 11) Sep 21, 2023
@olamy olamy merged commit 0a79d1e into jenkinsci:develop Sep 21, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants