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.

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

Colours updated for HTML output. #433

Merged
merged 2 commits into from
Oct 8, 2017
Merged

Colours updated for HTML output. #433

merged 2 commits into from
Oct 8, 2017

Conversation

jolkdarr
Copy link
Contributor

@jolkdarr jolkdarr commented Oct 6, 2017

CSS simpler, rendering cleaner, report clearer.

@iloveeclipse
Copy link
Member

Can you please attach picture before/after? I also see thatyou have removed the link to the project - why?

@jolkdarr
Copy link
Contributor Author

jolkdarr commented Oct 6, 2017

The link is dropped in my personal version of the file. => will be restored.
ASAP, I will provide two HTML reports that show off the improvement (according to me ;) ).

@jolkdarr
Copy link
Contributor Author

jolkdarr commented Oct 7, 2017

Hello
I join two reports for the Javolution library:

Copy link
Member

@iloveeclipse iloveeclipse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new report looks better, thanks.

@@ -141,10 +142,11 @@
<body>

<h1>SpotBugs Report</h1>
<p>Produced using <a href="https://spotbugs.github.io">SpotBugs </a> <xsl:value-of select="/BugCollection/@version"/>.</p>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update your commit and restore the link.

@iloveeclipse iloveeclipse merged commit 4f6e2c0 into spotbugs:master Oct 8, 2017
KengoTODA added a commit to KengoTODA/spotbugs that referenced this pull request Oct 9, 2017
@KengoTODA KengoTODA added this to the SpotBugs 3.1.0 milestone Oct 9, 2017
@KengoTODA KengoTODA mentioned this pull request Oct 9, 2017
iloveeclipse pushed a commit that referenced this pull request Oct 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants