[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Add SvnReporter details

From: <kfogel_at_collab.net>
Date: 2006-01-29 14:55:38 CET

"S.Ramaswamy" <srsy70@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
Received on Sun Jan 29 16:33:36 2006

This is an archived mail posted to the Subversion Dev mailing list.