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

Automated changelog for 2.335 #4909

Merged
merged 1 commit into from
Feb 16, 2022
Merged

Conversation

jenkins-infra-changelog-generator[bot]
Copy link
Contributor

@jenkins-infra-changelog-generator jenkins-infra-changelog-generator bot requested a review from a team as a code owner February 9, 2022 13:56
@probot-autolabeler probot-autolabeler bot added the changelog Jenkins changelogs label Feb 9, 2022
@jenkins-infra-changelog-generator jenkins-infra-changelog-generator bot force-pushed the automated-weekly-changelog/2.335 branch from 9ee710e to 00c09bc Compare February 9, 2022 17:24
@jenkins-infra-changelog-generator jenkins-infra-changelog-generator bot force-pushed the automated-weekly-changelog/2.335 branch 5 times, most recently from e88ef3a to eb682df Compare February 9, 2022 18:06
@jenkins-infra-changelog-generator jenkins-infra-changelog-generator bot force-pushed the automated-weekly-changelog/2.335 branch 2 times, most recently from ea78b17 to 7e851a3 Compare February 10, 2022 16:37
@jenkins-infra-changelog-generator jenkins-infra-changelog-generator bot force-pushed the automated-weekly-changelog/2.335 branch from 7e851a3 to 5ec3844 Compare February 10, 2022 22:39
@jenkins-infra-changelog-generator jenkins-infra-changelog-generator bot force-pushed the automated-weekly-changelog/2.335 branch from 5ec3844 to 9dbc039 Compare February 11, 2022 19:57
@jenkins-infra-changelog-generator jenkins-infra-changelog-generator bot force-pushed the automated-weekly-changelog/2.335 branch 5 times, most recently from 1e14ae0 to 422a429 Compare February 12, 2022 15:45
@jenkins-infra-changelog-generator jenkins-infra-changelog-generator bot force-pushed the automated-weekly-changelog/2.335 branch 2 times, most recently from 0184033 to 48dfcb1 Compare February 12, 2022 16:53
@jenkins-infra-changelog-generator jenkins-infra-changelog-generator bot force-pushed the automated-weekly-changelog/2.335 branch 2 times, most recently from 092ec9b to f06ff91 Compare February 13, 2022 20:02
Copy link
Member

@dheerajodha dheerajodha left a comment

Choose a reason for hiding this comment

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

Some language-related suggestions.

- NotMyFault
pr_title: "[JENKINS-67742] Replace computer-flash gif"
message: |-
Replace computer-flash gif icon with hourglass
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Replace computer-flash gif icon with hourglass
Replace the computer-flash GIF icon with the hourglass icon.

Copy link
Contributor

Choose a reason for hiding this comment

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

Applied to jenkinsci/jenkins#6254 pull request description. Thanks!

- basil
pr_title: Adapt to jenkinsci/remoting#490
message: |-
Remove support for <code>RoleChecker#check(RoleSensitive)</code> calls added in Jenkins 2.319 again. All remoting <code>Callable</code> implementations need to perform an actual role check as documented at https://www.jenkins.io/doc/developer/security/remoting-callables/
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Remove support for <code>RoleChecker#check(RoleSensitive)</code> calls added in Jenkins 2.319 again. All remoting <code>Callable</code> implementations need to perform an actual role check as documented at https://www.jenkins.io/doc/developer/security/remoting-callables/
Remove support for <code>RoleChecker#check(RoleSensitive)</code> calls which were added again in Jenkins 2.319. All remoting <code>Callable</code> implementations need to perform an actual role check as documented at https://www.jenkins.io/doc/developer/security/remoting-callables/

I tried to understand the context, but I might be wrong.

Copy link
Contributor

@MarkEWaite MarkEWaite Feb 15, 2022

Choose a reason for hiding this comment

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

Applied to the pull request description of jenkinsci/jenkins#5901

- janfaracik
pr_title: Add support for Ionicons (symbols) to <l:Icon> and <l:task> components
message: |-
Several icons across Jenkins have been updated - the build icon, the delete icon, the settings icon, the about icon and the plugin icon
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Several icons across Jenkins have been updated - the build icon, the delete icon, the settings icon, the about icon and the plugin icon
Several icons across Jenkins have been updated - the build icon, the delete icon, the settings icon, the about icon, and the plugin icon.

Copy link
Contributor

Choose a reason for hiding this comment

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

- janfaracik
pr_title: Polish forms components and split out CSS files
message: |-
Minor form and spacing changes
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Minor form and spacing changes
Minor form and spacing changes.

Copy link
Contributor

Choose a reason for hiding this comment

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

- NotMyFault
pr_title: "[JENKINS-67609] Deduplicate help button question mark"
message: |-
Set the question mark on the new help button only once so that it is not shown twice when using different themes.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Set the question mark on the new help button only once so that it is not shown twice when using different themes.
Render the question mark on the new help button only once, even while using different themes.
Suggested change
Set the question mark on the new help button only once so that it is not shown twice when using different themes.
Render the question mark on the new help button only once so that it is not shown twice while using different themes.

Maybe?

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Member

@dheerajodha dheerajodha left a comment

Choose a reason for hiding this comment

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

Some suggestions, that can be incorporated after the release.

- dependabot[bot]
- basil
pr_title: 'Winstone 5.22: Bump Jetty from 9.4.43.v20210629 to 9.4.45.v20220203'
message: |-
Copy link
Member

Choose a reason for hiding this comment

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

Not sure how this needs to be formatted in order to look as intended in the changelog.
Plus, this is a "developer"-category entry, so needs to be added as the last entry

Copy link
Contributor

@MarkEWaite MarkEWaite Feb 15, 2022

Choose a reason for hiding this comment

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

Will need to be moved into the correct location and have additional links after the release is complete. Attempted some formatting changes in jenkinsci/jenkins#6262, more likely needed after the release

@MarkEWaite action item after the weekly release has been delivered.

Append to log file rather than truncating it (https://github.com/jenkinsci/winstone/pull/200)
Write log file and access logs in UTF-8 encoding rather than default encoding (https://github.com/jenkinsci/winstone/pull/200)
Jetty 9.4.45 changelog: https://github.com/eclipse/jetty.project/releases/tag/jetty-9.4.45.v20220203
- &1
Copy link
Member

Choose a reason for hiding this comment

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

Needs to be removed

Copy link
Contributor

Choose a reason for hiding this comment

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

@MarkEWaite action item after release is available

pr_title: Modernise form components
message: |-
Modernise form components.
- *1
Copy link
Member

Choose a reason for hiding this comment

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

Needs to be removed.

Copy link
Contributor

Choose a reason for hiding this comment

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

@MarkEWaite action item after release is available

pr_title: "[JENKINS-67609] Deduplicate help button question mark"
message: |-
Set the question mark on the new help button only once so that it is not shown twice when using different themes.
- type: bug
Copy link
Member

Choose a reason for hiding this comment

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

9 rfe and 6 bug type entries, let's try to display them alternately.

Copy link
Contributor

Choose a reason for hiding this comment

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

I need to discuss the concept of "alternating" with Daniel Beck to assure that I understand his intention there. We've been grouping rfe as one group and bug as another group most recently and it may be time that I understand the intent of the alternation comment in the style guide. Thanks for the reminder!

Copy link
Member

Choose a reason for hiding this comment

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

@MarkEWaite Any updates on this?

@MarkEWaite MarkEWaite merged commit a8e7a4b into master Feb 16, 2022
@MarkEWaite MarkEWaite deleted the automated-weekly-changelog/2.335 branch February 16, 2022 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog Jenkins changelogs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants