-
Notifications
You must be signed in to change notification settings - Fork 105
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
forward compatibility for core dialogs #286
Conversation
Change jenkinsci/jenkins#7938 released with 2.415 introduces native dialogs as replacement of browser dialogs. Make the tests work with both variants.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks very much!
@rsandell could you label this pull request as If the merge can't happen before Monday, I'll add a temporary exclusion to the bom tests so that it will ignore the test failures. |
Tests fail with Jenkins 2.415 PR jenkinsci/config-file-provider-plugin#286 Confirmed that PR works, do not delay BOM release for test fix
Tests fail with Jenkins 2.415 PR jenkinsci/config-file-provider-plugin#286 Confirmed that PR works, do not delay BOM release for test fix
Fixed issue jenkinsci#2274 with pull request jenkinsci#2276 jenkinsci/config-file-provider-plugin#286 fixed it thanks to @mawinter69 and @rsandell. This reverts commit 8fe50be.
Fixed issue #2274 with pull request #2276 jenkinsci/config-file-provider-plugin#286 fixed it thanks to @mawinter69 and @rsandell. This reverts commit 8fe50be.
Change jenkinsci/jenkins#7938 released with 2.415 introduces native dialogs as replacement of browser dialogs. Make the tests work with both variants.
Testing done
run tests against 2.414 and 2.415 and both succeeded
Submitter checklist