Skip to content
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

DefaultCFLintMarshaller doesn't have any of the CFLint XML improvements in XMLOutput #330

Closed
TheRealAgentK opened this issue Jul 2, 2017 · 1 comment
Closed

DefaultCFLintMarshaller doesn't have any of the CFLint XML improvements in XMLOutput #330

TheRealAgentK opened this issue Jul 2, 2017 · 1 comment
Assignees
@TheRealAgentK
Labels
bug
Milestone
1.2.0

Comments

@TheRealAgentK
Copy link
Collaborator

TheRealAgentK commented Jul 2, 2017 •
edited
Loading

Just realised that when XML is written in non-findbugs style all the improvements to counts and timestamps we made are not output.

Reason: it's using a class called DefaultCFLintMarshaller and not XMLOutput.

Related to #323 issue 4
@TheRealAgentK TheRealAgentK self-assigned this Jul 2, 2017
@TheRealAgentK TheRealAgentK added this to the 1.2.0 milestone Jul 2, 2017
@TheRealAgentK
Copy link
Collaborator Author

Fixed!

TheRealAgentK added a commit to TheRealAgentK/CFLint that referenced this issue Jul 2, 2017
@TheRealAgentK
cflint#330 - Added files counts and timestamp to DefaultCFLintResultM…
6250929 
…arshaller to make it functionally equivalent with XMLOutput
@TheRealAgentK TheRealAgentK mentioned this issue Jul 2, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TheRealAgentK TheRealAgentK

Labels
bug
Projects
None yet
Milestone
1.2.0
Development

No branches or pull requests
1 participant
@TheRealAgentK