SA1625 does not take xml into account

[appel1]

I don't think the following should trigger a violation.

/// <typeparam name="TFirst">The type of elements of <paramref name="first"/>.</typeparam>
/// <typeparam name="TSecond">The type of elements of <paramref name="second"/>.</typeparam>

[sharwell]

I agree.

I wouldn't mind including this in 1.0 if we have to release another RC build, but I don't want to delay 1.0 stable for just this issue. I'm tentatively marking this for 1.0 so we don't accidentally overlook it.

/cc @pdelvo

[martincostello]

Another test case for this issue, except with <param> instead of <typeparam>:

/// <param name="dependencyOne">The optional <see cref="IDependencyOne"/> to use.</param>
/// <param name="dependencyTwo">The optional <see cref="IDependencyTwo"/> to use.</param>