alm: clean up warnings
- [x] Make `Git.Utilities.EnumerateParentProcesses` static. - [x] Supress "DefaultParametersShouldNotBeUsed" warnings on `TargetUri`.
This commit is contained in:
Родитель
daebd429b3
Коммит
7119df3ff1
|
@ -92,7 +92,7 @@ namespace Microsoft.Alm.Authentication.Git
|
|||
return false;
|
||||
}
|
||||
|
||||
internal IEnumerable<Win32.ProcessEntry32> EnumerateParentProcesses()
|
||||
internal static IEnumerable<Win32.ProcessEntry32> EnumerateParentProcesses()
|
||||
{
|
||||
var processTable = new Dictionary<uint, uint>();
|
||||
var processEntries = new Dictionary<uint, Win32.ProcessEntry32>();
|
||||
|
|
|
@ -230,6 +230,7 @@ namespace Microsoft.Alm.Authentication
|
|||
/// The new URL used for all queries if not `<see langword="null"/>`; otherwise is unchanged from `<seealso cref="ActualUri"/>`.
|
||||
/// </param>
|
||||
/// <exception cref="ArgumentException">When all arguments are `<see langword="null"/>`.</exception>
|
||||
[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Design", "CA1026:DefaultParametersShouldNotBeUsed")]
|
||||
public TargetUri CreateWith(string queryUrl = null, string proxyUrl = null, string actualUrl = null)
|
||||
{
|
||||
if (queryUrl is null && proxyUrl is null && actualUrl is null)
|
||||
|
@ -255,6 +256,7 @@ namespace Microsoft.Alm.Authentication
|
|||
/// The new `<seealso cref="Uri"/>` used for all queries if not `<see langword="null"/>`; otherwise is unchanged from `<seealso cref="ActualUri"/>`.
|
||||
/// </param>
|
||||
/// <exception cref="ArgumentException">When all arguments are `<see langword="null"/>`.</exception>
|
||||
[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Design", "CA1026:DefaultParametersShouldNotBeUsed")]
|
||||
public TargetUri CreateWith(Uri queryUri = null, Uri proxyUri = null, Uri actualUri = null)
|
||||
{
|
||||
if (queryUri is null && proxyUri is null && actualUri is null)
|
||||
|
|
Загрузка…
Ссылка в новой задаче