-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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.
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
Minor improvements to SpotBugs and Dependency Check parsers #4249
Minor improvements to SpotBugs and Dependency Check parsers #4249
Conversation
The report contains some HTML code into it that was not being proper parsed. Now, using BeautifulSoup, we can work with tags and deliver a more precise report. Enhancements: - Handles leading blankspaces, multiple breaklines, bold text and inline/multiline code. - A more clear description. - References moved to where they belong with links that work.
Thanks. Are there any unit tests / samples you could add that "prove" the new html parsing part is working correctly? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't introduce BS instead of html2text
that we use in a lot of parsers
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
I'm not sure if these unittests for the HTML parsing are good enough. Let me know what you guys think ;) |
No need for them to be there (it was actually adding more breaklines for some reason).
dbaa441
to
bc34fee
Compare
41f90dd
to
bb2e4c2
Compare
SpotBugs
The report contains some HTML code that was not being properly parsed.
Now, using BeautifulSoup, we can work with tags and deliver a more precise report.
Enhancements:
Dependency Check