• Type: Task
• Resolution: Fixed
• Priority: Minor
• Component/s: warnings-plugin
• Labels:

  • DEVTOOLS

[JENKINS-17791] Replace Pep8 parser with native parser

Ulli Hafner created issue - 2013-04-30 06:23

Ulli Hafner made changes - 2013-04-30 06:23

Link

New: This issue is blocking JENKINS-17786 [ JENKINS-17786 ]

Ulli Hafner made changes - 2013-04-30 06:24

Description

Original: {code}         parsers.add(new ViolationsAdapter(new JcReportParser(),                 Messages._Warnings_JCReport_ParserName(),                 Messages._Warnings_JCReport_LinkName(),                 Messages._Warnings_JCReport_TrendName())); {code}

New: {code}         parsers.add(new ViolationsAdapter(new Pep8Parser(),                 Messages._Warnings_Pep8_ParserName(),                 Messages._Warnings_Pep8_LinkName(),                 Messages._Warnings_Pep8_TrendName())); {code}

Marvin Schütz made changes - 2013-05-14 14:09

Assignee

Original: Ulli Hafner [ drulli ]

New: Marvin Schütz [ spiky91 ]

Collapse comment: SCM/JIRA link daemon added a comment - 2013-06-18 14:38

SCM/JIRA link daemon added a comment - 2013-06-18 14:38

Code changed in jenkins
User: Ulli Hafner
Path:
.classpath
.settings/org.eclipse.core.resources.prefs
src/main/java/hudson/plugins/warnings/parser/Pep8Parser.java
src/main/java/hudson/plugins/warnings/parser/ViolationsRegistry.java
src/test/java/hudson/plugins/warnings/parser/Pep8ParserTest.java
src/test/resources/hudson/plugins/warnings/parser/pep8Test.txt
http://jenkins-ci.org/commit/warnings-plugin/233085dae99708d2e4ddd17d8df3e52286d3b2a6
Log:
Merge pull request #25 from habast/jenkins-Pep8Parser

[FIXED JENKINS-17791] Added native Pep8 parser.

Compare: https://github.com/jenkinsci/warnings-plugin/compare/d958f11ad36d...233085dae997

Expand comment: SCM/JIRA link daemon added a comment - 2013-06-18 14:38

SCM/JIRA link daemon added a comment - 2013-06-18 14:38 Code changed in jenkins User: Ulli Hafner Path: .classpath .settings/org.eclipse.core.resources.prefs src/main/java/hudson/plugins/warnings/parser/Pep8Parser.java src/main/java/hudson/plugins/warnings/parser/ViolationsRegistry.java src/test/java/hudson/plugins/warnings/parser/Pep8ParserTest.java src/test/resources/hudson/plugins/warnings/parser/pep8Test.txt http://jenkins-ci.org/commit/warnings-plugin/233085dae99708d2e4ddd17d8df3e52286d3b2a6 Log: Merge pull request #25 from habast/jenkins-Pep8Parser [FIXED JENKINS-17791] Added native Pep8 parser. Compare: https://github.com/jenkinsci/warnings-plugin/compare/d958f11ad36d...233085dae997

SCM/JIRA link daemon made changes - 2013-06-18 14:38

Resolution		New: Fixed [ 1 ]
Status	Original: Open [ 1 ]	New: Resolved [ 5 ]

R. Tyler Croy made changes - 2016-07-25 23:55

Workflow

Original: JNJira [ 148971 ]

New: JNJira + In-Review [ 192968 ]