You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by "S.Ramaswamy" <sr...@gmail.com> on 2006/01/25 18:44:17 UTC

[PATCH] Add SvnReporter details

[[[
Update project_links.html.

* www/project_links.html: Add SvnReporter to list of miscellaneous utilities.
]]]

Re: [PATCH] Add SvnReporter details

Posted by kf...@collab.net.
"S.Ramaswamy" <sr...@gmail.com> writes:
> [[[
> Update project_links.html.
> 
> * www/project_links.html: Add SvnReporter to list of miscellaneous utilities.
> ]]]

Thanks, Ram!  Committed in 18285.

Next time, can you attach as text/plain, not application/octet-stream?
That way people can view the patch inline in their mailreaders.  Also,
a minor nit:

> Index: www/project_links.html
> ===================================================================
> --- www/project_links.html	(revision 18224)
> +++ www/project_links.html	(working copy)
> @@ -682,6 +682,11 @@
>          <a href="http://confluence.atlassian.com/display/JIRAEXT/JIRA+Subversion+plugin"
>                  >http://confluence.atlassian.com/display/JIRAEXT/JIRA+Subversion+plugin</a>
>        </p></li>
> +      <li><p><b>SvnReporter</b> - Python script to generate various reports
> +      from Subversion commits.<br/>
> +      <a href="http://www.calins.ch/software/SvnReporter.html">
> +      http://www.calins.ch/software/SvnReporter.html</a>
> +      </p></li>
>      </ul>
>  
>  </div>

The ">" after the href should be on the next line, as the other items
in that list do.  Otherwise, you include a whitespace in the
hyperlinked text.

Best,
-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org