Skip to content

Conversation

@joseblas
Copy link

@joseblas joseblas commented Oct 1, 2020

No description provided.

@jglick
Copy link
Member

jglick commented Oct 5, 2020

Is this superseded by jenkinsci/bom#315?

@timja
Copy link
Member

timja commented Oct 16, 2020

Is this superseded by jenkinsci/bom#315?

looks like not:
jenkinsci/bom#294 (comment)

@timja
Copy link
Member

timja commented Oct 17, 2020

Tested this in jenkinsci/bom#294 and it works...

@jglick
Copy link
Member

jglick commented Oct 19, 2020

Please merge with master @joseblas.


if (failed && config.isFailOnError()) {
throw new AbortException("Execution was aborted due to the failure in a plugin test (-failOnError is set)");
throw new AbortException("Execution was aborted due to the failure in a plugin test (-failOnError is set)");
Copy link
Member

Choose a reason for hiding this comment

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

unnecessary whitespace change, and adds tabs to a file that uses spaces


return report;
}
}
Copy link
Member

Choose a reason for hiding this comment

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

unnecessary whitespace change, and adds tabs to a file that uses spaces

File pluginCheckoutDir = new File(config.workDirectory.getAbsolutePath() + File.separator + plugin.name + File.separator);

try {
try {
Copy link
Member

Choose a reason for hiding this comment

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

unnecessary whitespace change, and adds tabs to a file that uses spaces

@joseblas joseblas closed this Nov 4, 2020
@timja
Copy link
Member

timja commented Nov 4, 2020

@joseblas why was this closed? we're relying on it in bom

@jglick
Copy link
Member

jglick commented Nov 4, 2020

If we are consuming it, it needs to remain open.

@jglick
Copy link
Member

jglick commented Mar 11, 2021

Replaced by #276.

@jglick jglick closed this Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants