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 test dependency on multiple-scms plugin #997

Merged
merged 1 commit into from
Nov 4, 2020

Conversation

timja
Copy link
Member

@timja timja commented Nov 4, 2020

Relates to https://issues.jenkins-ci.org/browse/JENKINS-64136 although doesn't fix it

Without these changes you can't run this plugin using hpi:run on 2.264.

At least 3.6 for promoted builds is needed to pickup a tables to divs fix

and multiple-scms is deprecated and abandoned, it's very broken... (including broken on tables to div although broken before that as well)

@timja timja force-pushed the make-plugin-testable-hpi-run branch from 606753b to f4ab92a Compare November 4, 2020 21:05
@MarkEWaite MarkEWaite added the bugfix Fixes a bug - used by Release Drafter label Nov 4, 2020
@MarkEWaite MarkEWaite changed the title Make plugin testable with tables to divs Remove test dependency on multiple-scms plugin Nov 4, 2020
@MarkEWaite
Copy link
Contributor

At least 3.6 for promoted builds is needed to pickup a tables to divs fix

I had kept promoted builds plugin at an older version for fear of forcing users to a newer version that might have compatibility issues. Several releases of the promoted builds plugin had issues that prevented user upgrades.

However, the current stats show that the users of promoted-builds plugin that are on Jenkins 2.204.1 or newer (minimum version supported by git plugin) have almost all upgraded to promoted builds plugin 3.5. If they've upgraded to 3.5, then they've already upgraded past the versions that had issues.

@MarkEWaite MarkEWaite merged commit 088b4f6 into jenkinsci:master Nov 4, 2020
@timja timja deleted the make-plugin-testable-hpi-run branch November 4, 2020 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Fixes a bug - used by Release Drafter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants