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

Move old bug count logic for output into cflintstats #299

Closed
TheRealAgentK opened this issue Jun 25, 2017 · 1 comment
Closed

Move old bug count logic for output into cflintstats #299

TheRealAgentK opened this issue Jun 25, 2017 · 1 comment
Assignees
@TheRealAgentK
Labels
enhancement
Milestone
1.2.0

Comments

@TheRealAgentK
Copy link
Collaborator

Todo for myself as a flow-on from #278.

Code's working fine, but the existing logic for tracking which rule was triggered how many times should probably be moved into CFLintStats so that all the stat-related info for output is just in one place.
@TheRealAgentK TheRealAgentK self-assigned this Jun 25, 2017
@TheRealAgentK TheRealAgentK added this to the 1.2.0 milestone Jun 27, 2017
TheRealAgentK added a commit to TheRealAgentK/CFLint that referenced this issue Jun 30, 2017
@TheRealAgentK
cflint#299 - Refactoring BugCount into CFLintStat for output generation
ca589ff
TheRealAgentK added a commit to TheRealAgentK/CFLint that referenced this issue Jun 30, 2017
@TheRealAgentK
cflint#299 - Moving bug counts from individual output routines into a…
b3dabb0 
… central place-pre output. Adjusting tests, added some new tests for JSON/XML output stats
@TheRealAgentK
Copy link
Collaborator Author

PR #320

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TheRealAgentK TheRealAgentK

Labels
enhancement
Projects
None yet
Milestone
1.2.0
Development

No branches or pull requests
1 participant
@TheRealAgentK