-
Notifications
You must be signed in to change notification settings - Fork 36
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 obsolete classes #34
Conversation
* @author Oleg Nenashev | ||
*/ | ||
//TODO: 1.567 provides a replacement | ||
public class DecoratedLauncher extends Launcher { |
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.
This class is still a part of the public API though I do not see any usages in GitHub. I am fine with removing it
@oleg-nenashev is there anything to do or can it be merged? |
I believe this is okay.
This plugin just needs a maintainer to get it released
…On Thu, Mar 7, 2019, 23:18 René Scheibe ***@***.***> wrote:
@oleg-nenashev <https://github.com/oleg-nenashev> is there anything to do
or can it be merged?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#34 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AC3IoOw84kaQqDlHW5bpOMNIrIFcRruMks5vUZAfgaJpZM4bBmil>
.
|
@oleg-nenashev could you at least merge it? |
@recampbell do you maintain this plugin? At least you are listed as |
No, the plugin has no active maintainer. I can deliver changes on-demand tho |
This one will not hurt anyway |
Thanks. |
JenkinsHelper
andDecoratedLauncher
are not required anymore.