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.

Sign up for GitHub

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

Jump to bottom

[JENKINS-72054] Add an option to skip the time-consuming post-processing step #1651

Merged

uhafner merged 6 commits into main from skipPostProcessing

Jan 9, 2024

Merged

[JENKINS-72054] Add an option to skip the time-consuming post-processing step #1651

Add documentation of new field in all steps.

ci.jenkins.io / CPD succeeded Jan 9, 2024 in 0s

7 new issues, 304 total

Total	New	Outstanding	Fixed	Trend
304	7	297	6	👎

Reference build: Plugins » warnings-ng-plugin » main #25

Details

Severity distribution of new issues

Error	Warning High	Warning Normal	Warning Low
0	1	3	3

Annotations

Check warning on line 221 in plugin/src/main/java/io/jenkins/plugins/analysis/core/steps/ScanForIssuesStep.java

ci-jenkins-io / CPD

CPD

HIGH:
Found duplicated code.

Raw output


            <pre><code>} &#64;CheckForNull public String getSourceCodeEncoding() { return sourceCodeEncoding; } /** * Sets the encoding to use to read source files. * * &#64;param sourceCodeEncoding * the encoding, e.g. &#34;ISO-8859-1&#34; */ &#64;DataBoundSetter public void setSourceCodeEncoding(final String sourceCodeEncoding) { this.sourceCodeEncoding &#61; sourceCodeEncoding; } public String getSourceDirectory() { return sourceDirectory; } /** * Sets the path to the folder that contains the source code. If not relative and thus not part of the workspace * then this folder needs to be added in Jenkins global configuration. * * &#64;param sourceDirectory * a folder containing the source code */ &#64;DataBoundSetter public void setSourceDirectory(final String sourceDirectory) { this.sourceDirectory &#61; sourceDirectory; } /** * Sets the paths to the directories that contain the source code. If not relative and thus not part of the * workspace then these directories need to be added in Jenkins global configuration to prevent accessing of * forbidden resources. * * &#64;param sourceDirectories * directories containing the source code */ &#64;DataBoundSetter public void setSourceDirectories(final List&lt;SourceCodeDirectory&gt; sourceDirectories) { this.sourceDirectories &#61; new HashSet&lt;&gt;(sourceDirectories); } public List&lt;SourceCodeDirectory&gt; getSourceDirectories() { return new ArrayList&lt;&gt;(sourceDirectories); } private List&lt;SourceCodeDirectory&gt; getAllSourceDirectories() {<!-- --></code></pre>

Check warning on line 218 in plugin/src/main/java/io/jenkins/plugins/analysis/core/steps/ScanForIssuesStep.java

ci-jenkins-io / CPD

CPD

NORMAL:
Found duplicated code.

Raw output


            <pre><code>} &#64;CheckForNull public String getSourceCodeEncoding() { return sourceCodeEncoding; } /** * Sets the encoding to use to read source files. * * &#64;param sourceCodeEncoding * the encoding, e.g. &#34;ISO-8859-1&#34; */ &#64;DataBoundSetter public void setSourceCodeEncoding(final String sourceCodeEncoding) { this.sourceCodeEncoding &#61; sourceCodeEncoding; } public String getSourceDirectory() { return sourceDirectory; } /** * Sets the path to the directory that contains the source code. If not relative and thus not part of the workspace * then this directory needs to be added in Jenkins global configuration to prevent accessing of forbidden resources. * * &#64;param sourceDirectory * directory containing the source code */ &#64;DataBoundSetter public void setSourceDirectory(final String sourceDirectory) { this.sourceDirectory &#61; sourceDirectory; } /** * Sets the paths to the directories that contain the source code. If not relative and thus not part of the workspace * then these directories need to be added in Jenkins global configuration to prevent accessing of forbidden resources. * * &#64;param sourceDirectories * directories containing the source code */ &#64;DataBoundSetter public void setSourceDirectories(final List&lt;SourceCodeDirectory&gt; sourceDirectories) { this.sourceDirectories &#61; new HashSet&lt;&gt;(sourceDirectories); } public List&lt;SourceCodeDirectory&gt; getSourceDirectories() { return new ArrayList&lt;&gt;(sourceDirectories); }</code></pre>

Check warning on line 119 in plugin/src/test/java/io/jenkins/plugins/analysis/warnings/steps/ModuleDetectorITest.java

ci-jenkins-io / CPD

CPD

LOW:
Found duplicated code.

Raw output


            <pre><code>void shouldShowModulesForVariousModulesDetectedForOsgiMavenAndAntInTheHtmlOutput() { String[] workspaceFiles &#61; { BUILD_FILE_PATH &#43; ANT_BUILD_FILE_LOCATION &#43; &#34;build.xml&#34;, BUILD_FILE_PATH &#43; ANT_BUILD_FILE_LOCATION &#43; &#34;m1/build.xml&#34;, BUILD_FILE_PATH &#43; MAVEN_BUILD_FILE_LOCATION &#43; &#34;pom.xml&#34;, BUILD_FILE_PATH &#43; MAVEN_BUILD_FILE_LOCATION &#43; &#34;m1/pom.xml&#34;, BUILD_FILE_PATH &#43; MAVEN_BUILD_FILE_LOCATION &#43; &#34;m2/pom.xml&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;m1/META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;m2/META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;m3/META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;plugin.properties&#34;};</code></pre>

Check warning on line 145 in plugin/src/test/java/io/jenkins/plugins/analysis/warnings/steps/ModuleDetectorITest.java

ci-jenkins-io / CPD

CPD

LOW:
Found duplicated code.

Raw output


            <pre><code>void shouldShowModulesForVariousModulesDetectedForOsgiMavenAndAntInTheHtmlOutput() { String[] workspaceFiles &#61; { BUILD_FILE_PATH &#43; ANT_BUILD_FILE_LOCATION &#43; &#34;build.xml&#34;, BUILD_FILE_PATH &#43; ANT_BUILD_FILE_LOCATION &#43; &#34;m1/build.xml&#34;, BUILD_FILE_PATH &#43; MAVEN_BUILD_FILE_LOCATION &#43; &#34;pom.xml&#34;, BUILD_FILE_PATH &#43; MAVEN_BUILD_FILE_LOCATION &#43; &#34;m1/pom.xml&#34;, BUILD_FILE_PATH &#43; MAVEN_BUILD_FILE_LOCATION &#43; &#34;m2/pom.xml&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;m1/META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;m2/META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;m3/META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;plugin.properties&#34;};</code></pre>

Check warning on line 240 in plugin/src/test/java/io/jenkins/plugins/analysis/warnings/steps/ModuleDetectorITest.java

ci-jenkins-io / CPD

CPD

LOW:
Found duplicated code.

Raw output


            <pre><code>void shouldShowModulesForVariousModulesDetectedForOsgiMavenAndAntInTheHtmlOutput() { String[] workspaceFiles &#61; { BUILD_FILE_PATH &#43; ANT_BUILD_FILE_LOCATION &#43; &#34;build.xml&#34;, BUILD_FILE_PATH &#43; ANT_BUILD_FILE_LOCATION &#43; &#34;m1/build.xml&#34;, BUILD_FILE_PATH &#43; MAVEN_BUILD_FILE_LOCATION &#43; &#34;pom.xml&#34;, BUILD_FILE_PATH &#43; MAVEN_BUILD_FILE_LOCATION &#43; &#34;m1/pom.xml&#34;, BUILD_FILE_PATH &#43; MAVEN_BUILD_FILE_LOCATION &#43; &#34;m2/pom.xml&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;m1/META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;m2/META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;m3/META-INF/MANIFEST.MF&#34;, BUILD_FILE_PATH &#43; OSGI_BUILD_FILE_LOCATION &#43; &#34;plugin.properties&#34;};</code></pre>

Check warning on line 894 in plugin/src/main/java/io/jenkins/plugins/analysis/core/steps/RecordIssuesStep.java

ci-jenkins-io / CPD

CPD

NORMAL:
Found duplicated code.

Raw output


            <pre><code>} /** * Not used anymore. * * &#64;return {&#64;code true} if SCM forensics should be disabled * &#64;deprecated Forensics will be automatically skipped if the Forensics recorder is not activated. */ &#64;SuppressWarnings(&#34;PMD.BooleanGetMethodName&#34;) &#64;Deprecated public boolean getForensicsDisabled() { return false; } /** * Not used anymore. * * &#64;param forensicsDisabled * not used * * &#64;deprecated Forensics will be automatically skipped if the Forensics recorder is not activated. */ &#64;DataBoundSetter &#64;Deprecated public void setForensicsDisabled(final boolean forensicsDisabled) { // do nothing } /** * Returns whether post-processing of the issues should be disabled. * * &#64;return {&#64;code true} if post-processing of the issues should be disabled. */ public boolean isSkipPostProcessing() { return skipPostProcessing; } &#64;DataBoundSetter public void setSkipPostProcessing(final boolean skipPostProcessing) { this.skipPostProcessing &#61; skipPostProcessing; }</code></pre>

Check warning on line 170 in plugin/src/main/java/io/jenkins/plugins/analysis/core/steps/ScanForIssuesStep.java

ci-jenkins-io / CPD

CPD

NORMAL:
Found duplicated code.

Raw output


            <pre><code>} /** * Not used anymore. * * &#64;return {&#64;code true} if SCM forensics should be disabled * &#64;deprecated Forensics will be automatically skipped if the Forensics recorder is not activated. */ &#64;SuppressWarnings(&#34;PMD.BooleanGetMethodName&#34;) &#64;Deprecated public boolean getForensicsDisabled() { return false; } /** * Not used anymore. * * &#64;param forensicsDisabled * not used * * &#64;deprecated Forensics will be automatically skipped if the Forensics recorder is not activated. */ &#64;DataBoundSetter &#64;Deprecated public void setForensicsDisabled(final boolean forensicsDisabled) { // do nothing } /** * Returns whether post-processing of the issues should be disabled. * * &#64;return {&#64;code true} if post-processing of the issues should be disabled. */ public boolean isSkipPostProcessing() { return skipPostProcessing; } &#64;DataBoundSetter public void setSkipPostProcessing(final boolean skipPostProcessing) { this.skipPostProcessing &#61; skipPostProcessing; }</code></pre>

View more details on ci.jenkins.io