Add markewaite as a zoom plugin maintainer#5043
Open
MarkEWaite wants to merge 1 commit intojenkins-infra:masterfrom
Open
Add markewaite as a zoom plugin maintainer#5043MarkEWaite wants to merge 1 commit intojenkins-infra:masterfrom
MarkEWaite wants to merge 1 commit intojenkins-infra:masterfrom
Conversation
https://issues.jenkins.io/browse/JENKINS-76442 reports a class not found exception in the Zoom plugin due to a class loading issue with the Jackson 2 API plugin. The issue is confirmed resolved by pull request: * jenkinsci/zoom-plugin#23 When I submitted the pull request, I mentioned @ZeyuCai-ZOOM and @LukasLuZoom asking them to merge and release the pull request so that other users of the Zoom plugin could receive the fix. There has been no response to that request in the last week. Yesterday I sent email to the zoom_us email address from issues.jenkins.io. I would like to be added as a maintainer so that I can merge and release pull request: * jenkinsci/zoom-plugin#23 That pull request replaces several other pending pull requests on the plugin, including: * jenkinsci/zoom-plugin#22 * jenkinsci/zoom-plugin#21 * jenkinsci/zoom-plugin#20 * jenkinsci/zoom-plugin#15 * jenkinsci/zoom-plugin#14 * jenkinsci/zoom-plugin#9 I do not intend to add new features to the plugin. I plan to merge the one pull request, close the pull requests that it replaces, and take the necessary steps to switch the plugin to use automated releases. Testing done: * The issue reporter has confirmed that the pull request fixes the issue. The same technique has been used in a few other plugins affected by the same problem.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add markewaite as a zoom plugin maintainer
JENKINS-76442 reports a class not found exception in the Zoom plugin due to a class loading issue with the Jackson 2 API plugin. The issue is confirmed resolved by pull request:
When I submitted the pull request, I mentioned @ZeyuCai-ZOOM and @LukasLuZoom asking them to merge and release the pull request so that other users of the Zoom plugin could receive the fix. There has been no response to that request in the last week.
Yesterday I sent email to the zoom_us email address from issues.jenkins.io.
I would like to be added as a maintainer so that I can merge and release pull request:
That pull request replaces several other pending pull requests on the plugin, including:
I do not intend to add new features to the plugin. I plan to merge the one pull request, close the pull requests that it replaces, and take the necessary steps to switch the plugin to use automated releases.
Link to GitHub repository
https://github.com/jenkinsci/zoom-plugin
Existing team members to approve this pull request:
I would prefer that they merge and release the fix so that I don't need to become a maintainer of the plugin. However, if they don't merge and release the fix, then it is better that I become a maintainer instead of requiring multiple users to install the incremental build with the fix. Installing an incremental build is more complicated and the incremental build will only remain in our artifact repository for 3 months before it is deleted.
When modifying release permission
List the GitHub usernames of the users who should have commit permissions below:
@markewaiteThis is needed in order to cut releases of the plugin or component.
If you are modifying the release permission of your plugin or component, fill out the following checklist:
Release permission checklist (for submitters)
When enabling automated releases (cd: true)
Follow the documentation to ensure, your pull request is set up properly. Don't merge it yet.
In case of changes requested by the hosting team, an open PR facilitates future reviews, without derailing work across multiple PRs.
Link to the PR enabling CD in your plugin
CD checklist (for submitters)
Reviewer checklist
$pluginId Developersteam hasAdminpermissions while granting the access.@Wadeck) in this pull request. If an email contact is changed, wait for approval from the security officer.There are IRC Bot commands for it.