-
Improvement
-
Resolution: Fixed
-
Critical
Pipeline jobs appears to be the standard interface for Jenkins 2.0 and forward.
I think it requires implementing the 'SimpleBuildStep' class interface.
Similar issue with many other publisher plugins, eg:
https://github.com/jenkinsci/cobertura-plugin/issues/50#issuecomment-222680913
[JENKINS-35234] Support the new workflow/pipeline model
misieq You can always claim ownership of the plugin if the maintainer is not responsive. Just drop an email to the ML. Regarding the reviews, do not hesitate to ping people regularly. It's almost impossible to process all incoming notifications, but by increasing the frequesncy you increase chances that it gets processed. Sounds weird, I know 
Code changed in jenkins
User: piomis
Path:
pom.xml
src/main/java/hudson/plugins/sloccount/SloccountAreaRenderer.java
src/main/java/hudson/plugins/sloccount/SloccountBuildAction.java
src/main/java/hudson/plugins/sloccount/SloccountProjectAction.java
src/main/java/hudson/plugins/sloccount/SloccountPublisher.java
src/main/java/hudson/plugins/sloccount/SloccountResult.java
src/main/java/hudson/plugins/sloccount/dashboard/SloccountTablePortlet.java
src/main/java/hudson/plugins/sloccount/model/SloccountLanguageStatistics.java
src/main/java/hudson/plugins/sloccount/model/SloccountParser.java
src/main/java/hudson/plugins/sloccount/model/cloc/ClocFile.java
src/main/java/hudson/plugins/sloccount/model/cloc/ClocFiles.java
src/main/java/hudson/plugins/sloccount/model/cloc/ClocHeader.java
src/main/java/hudson/plugins/sloccount/model/cloc/ClocReport.java
src/main/java/hudson/plugins/sloccount/model/cloc/ClocTotal.java
src/main/java/hudson/plugins/sloccount/util/FileFinder.java
src/main/java/jenkins/plugins/sloccount/steps/SloccountPublisherStep.java
src/main/resources/hudson/plugins/sloccount/SloccountBuildAction/statistics.jelly
src/main/resources/hudson/plugins/sloccount/SloccountBuildAction/summary.jelly
src/main/resources/hudson/plugins/sloccount/SloccountProjectAction/floatingBox.jelly
src/main/resources/hudson/plugins/sloccount/SloccountProjectAction/jobMain.jelly
src/main/resources/hudson/plugins/sloccount/SloccountPublisher/config.jelly
src/main/resources/hudson/plugins/sloccount/SloccountResult/files.jelly
src/main/resources/hudson/plugins/sloccount/SloccountResult/folders.jelly
src/main/resources/hudson/plugins/sloccount/SloccountResult/index.jelly
src/main/resources/hudson/plugins/sloccount/SloccountResult/languages.jelly
src/main/resources/hudson/plugins/sloccount/SloccountResult/modules.jelly
src/main/resources/hudson/plugins/sloccount/dashboard/SloccountTablePortlet/portlet.jelly
src/main/resources/hudson/plugins/sloccount/dashboard/SloccountTablePortlet/table.jelly
src/main/resources/index.jelly
src/main/resources/jenkins/plugins/sloccount/steps/SloccountPublisherStep/config.jelly
src/main/resources/tabview/css.jelly
src/main/resources/tabview/distribution-graph.jelly
src/main/resources/tabview/main.jelly
src/test/java/jenkins/plugins/sloccount/steps/SloccountPublisherStepTest.java
http://jenkins-ci.org/commit/sloccount-plugin/784fc4905bb71e64cae78f74397c14a94cdf40f5
Log:
JENKINS-35234 Resolved issue - Support the new workflow/pipeline model
pom.xml
Added dependencies to workflow modules
Sloccount plugin now requires Jenkins 2.63
Refactored for new workflow (AbstractBuild and AbstractProject replaced
with Run<> and Job<>)
SloccountAreaRenderer.java
SloccountBuildAction.java
SloccountProjectAction.java
SloccountPublisher.java
SloccountResult.java
Solved NPE:
SloccountResult.java
Ignored FindBug errors:
SloccountAreaRenderer.java
Solved javadoc complains:
SloccountTablePortlet.java
SloccountLanguageStatistics.java
SloccountParser.java
ClocFile.java
ClocFiles.java
ClocHeader.java
ClocReport.java
ClocTotal.java
Updated API for new Jenkins:
FileFinder.java
Added escape-by-default in all jelly files:
index.jelly
SloccountTablePortlet/portlet.jelly
SloccountTablePortlet/table.jelly
SloccountBuildAction/statistics.jelly
SloccountBuildAction/summary.jelly
SloccountProjectAction/floatingBox.jelly
SloccountProjectAction/jobMain.jelly
SloccountPublisher/config.jelly
SloccountResult/files.jelly
SloccountResult/folders.jelly
SloccountResult/index.jelly
SloccountResult/languages.jelly
SloccountResult/modules.jelly
tabview/css.jelly
tabview/distribution-graph.jelly
tabview/main.jelly
Added SloccountPublisher step supporting workflow (with jelly file and
tests)
steps/SloccountPublisherStep.java
SloccountPublisherStep/config.jelly
test/SloccountPublisherStepTest.java
Code changed in jenkins
User: Piotr Mis
Path:
pom.xml
src/main/java/hudson/plugins/sloccount/SloccountBuildAction.java
src/main/java/hudson/plugins/sloccount/SloccountResult.java
http://jenkins-ci.org/commit/sloccount-plugin/6d883661b6aad00a8ac1e4aad1ce51e50e3f8694
Log:
JENKINS-35234 Resolved issue - Support the new workflow/pipeline model
pom.xml
- Corrected dependencies to master-pom, Jenkins version and workflow plugins
SloccountResult
- owner is not serializable so setter must be added and user when SloccountBuildAction is loaded
SloccountBuildAction.java
- Run<> must be passed to SloccountResult when build is loaded
Code changed in jenkins
User: Ivaïlo Marinkov
Path:
pom.xml
src/main/java/hudson/plugins/sloccount/SloccountAreaRenderer.java
src/main/java/hudson/plugins/sloccount/SloccountBuildAction.java
src/main/java/hudson/plugins/sloccount/SloccountChartBuilder.java
src/main/java/hudson/plugins/sloccount/SloccountProjectAction.java
src/main/java/hudson/plugins/sloccount/SloccountPublisher.java
src/main/java/hudson/plugins/sloccount/SloccountResult.java
src/main/java/hudson/plugins/sloccount/dashboard/SloccountTablePortlet.java
src/main/java/hudson/plugins/sloccount/model/SloccountLanguageStatistics.java
src/main/java/hudson/plugins/sloccount/model/SloccountParser.java
src/main/java/hudson/plugins/sloccount/model/cloc/ClocFile.java
src/main/java/hudson/plugins/sloccount/model/cloc/ClocFiles.java
src/main/java/hudson/plugins/sloccount/model/cloc/ClocHeader.java
src/main/java/hudson/plugins/sloccount/model/cloc/ClocReport.java
src/main/java/hudson/plugins/sloccount/model/cloc/ClocTotal.java
src/main/java/hudson/plugins/sloccount/util/FileFinder.java
src/main/java/jenkins/plugins/sloccount/steps/SloccountPublisherStep.java
src/main/resources/hudson/plugins/sloccount/SloccountBuildAction/statistics.jelly
src/main/resources/hudson/plugins/sloccount/SloccountBuildAction/summary.jelly
src/main/resources/hudson/plugins/sloccount/SloccountProjectAction/floatingBox.jelly
src/main/resources/hudson/plugins/sloccount/SloccountProjectAction/jobMain.jelly
src/main/resources/hudson/plugins/sloccount/SloccountPublisher/config.jelly
src/main/resources/hudson/plugins/sloccount/SloccountResult/files.jelly
src/main/resources/hudson/plugins/sloccount/SloccountResult/folders.jelly
src/main/resources/hudson/plugins/sloccount/SloccountResult/index.jelly
src/main/resources/hudson/plugins/sloccount/SloccountResult/languages.jelly
src/main/resources/hudson/plugins/sloccount/SloccountResult/modules.jelly
src/main/resources/hudson/plugins/sloccount/dashboard/SloccountTablePortlet/portlet.jelly
src/main/resources/hudson/plugins/sloccount/dashboard/SloccountTablePortlet/table.jelly
src/main/resources/index.jelly
src/main/resources/jenkins/plugins/sloccount/steps/SloccountPublisherStep/config.jelly
src/main/resources/tabview/css.jelly
src/main/resources/tabview/distribution-graph.jelly
src/main/resources/tabview/main.jelly
src/test/java/jenkins/plugins/sloccount/steps/SloccountPublisherStepTest.java
http://jenkins-ci.org/commit/sloccount-plugin/f7d1b6a6e355e8bc2af228fe37be4ac65f0ac591
Log:
Merge pull request #49 from ivaivalous/JENKINS-35234_pipeline_support
JENKINS-35234 pipeline support
Compare: https://github.com/jenkinsci/sloccount-plugin/compare/29c846b00a3f...f7d1b6a6e355
Ivaïlo Marinkov
added a comment - npiguet's changes have now been merged. I hope to proceed with the release process as soon as I get more familiar with how it happens (please bear with me, it's my first time).
Ivaïlo Marinkov
added a comment - npiguet 's changes have now been merged. I hope to proceed with the release process as soon as I get more familiar with how it happens (please bear with me, it's my first time). imarinkov Great, thanks for doing it! If there are any issues with the release flow, please do not hesitate to ping me. Should be more or less available starting from tomorrow
Ivaïlo Marinkov
added a comment - Thanks oleg_nenashev!
I've released v1.22 of the plugin and updated the wiki page. I have tested the pipeline step to work correctly.
It seems the description macro on the top is retrieved automatically from the repository. To add myself on the maintainers list, do I just change developers in pom.xml?
Ivaïlo Marinkov
added a comment - Thanks oleg_nenashev !
I've released v1.22 of the plugin and updated the wiki page . I have tested the pipeline step to work correctly.
It seems the description macro on the top is retrieved automatically from the repository. To add myself on the maintainers list, do I just change developers in pom.xml ? Ivaïlo Marinkov
added a comment - The changes are now available with the 1.22 release . Sorry for missing the question. You just need to update pom.xml and to release the new version
Looks that PR will not be merged any time soon.
Snapshow build can be found under following link: https://jenkins.ci.cloudbees.com/job/plugins/job/sloccount-plugin/91/hudson.plugins.sloccount$sloccount/
It has been used in my company production environment for nearly 2 months and no issues observed.