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

Update to a new analyzer package #6946

Merged
merged 5 commits into from
Jul 16, 2019
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion eng/Packages.Data.props
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
<ItemGroup>
<PackageReference Update="ApprovalTests" Version="3.0.22" />
<PackageReference Update="ApprovalUtilities" Version="3.0.22" />
<PackageReference Update="Azure.ClientSdk.Analyzers" Version="0.1.1-dev.20190711.4" />
<PackageReference Update="Azure.ClientSdk.Analyzers" Version="0.1.1-dev.20190716.1" />
pakrym marked this conversation as resolved.
Show resolved Hide resolved
<PackageReference Update="Azure.Core" Version="1.0.0-preview.6" />
<PackageReference Update="BenchmarkDotNet" Version="0.11.5" />
<PackageReference Update="FsCheck.Xunit" Version="2.14.0" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ public partial class ConfigurationClient
private readonly Uri _baseUri;
private readonly HttpPipeline _pipeline;


/// <summary>
/// Protected constructor to allow mocking
/// </summary>
Expand Down Expand Up @@ -304,7 +303,6 @@ public virtual async Task<Response<ConfigurationSetting>> UpdateAsync(string key
return await UpdateAsync(new ConfigurationSetting(key, value, label), cancellationToken).ConfigureAwait(false);
}


/// <summary>
/// Updates an existing <see cref="ConfigurationSetting"/> in the configuration store.
/// </summary>
Expand Down
1 change: 0 additions & 1 deletion sdk/core/Azure.Core/src/Shared/HashCodeBuilder.cs
Original file line number Diff line number Diff line change
Expand Up @@ -220,7 +220,6 @@ private static void Initialize(out uint v1, out uint v2, out uint v3, out uint v
public static uint RotateLeft(uint value, int offset)
=> (value << offset) | (value >> (64 - offset));


[MethodImpl(MethodImplOptions.AggressiveInlining)]
private static uint Round(uint hash, uint input)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,5 @@ public class IssuerBase
public DateTimeOffset Created { get; set; }
public DateTimeOffset Updated { get; set; }
public bool Enabled { get; set; }

}
}
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,6 @@ internal override void WriteProperties(Utf8JsonWriter json)
}
if (N != null)
{

json.WriteString(NPropertyNameBytes, Base64Url.Encode(N));
}
if (E != null)
Expand Down
2 changes: 1 addition & 1 deletion sdk/keyvault/Azure.Security.KeyVault.Keys/src/KeyBase.cs
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ protected void ParseId(string id)
}

internal override void WriteProperties(Utf8JsonWriter json) { }

internal override void ReadProperties(JsonElement json)
{
foreach(JsonProperty prop in json.EnumerateObject())
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,14 +14,14 @@ public class KeyClientOptions : ClientOptions
{
/// <summary>
/// The latest service version supported by this client library.
/// For more information, see
/// For more information, see
/// <see href="https://docs.microsoft.com/en-us/rest/api/keyvault/key-vault-versions"/>
/// </summary>
internal const ServiceVersion LatestVersion = ServiceVersion.V7_0;

/// <summary>
/// The versions of Azure Key Vault supported by this client
/// library.
/// library.
/// </summary>
public enum ServiceVersion
{
Expand All @@ -35,7 +35,7 @@ public enum ServiceVersion

/// <summary>
/// Gets the <see cref="ServiceVersion"/> of the service API used when
/// making requests. For more information, see
/// making requests. For more information, see
/// <see href="https://docs.microsoft.com/en-us/rest/api/keyvault/key-vault-versions"/>
/// </summary>
public ServiceVersion Version { get; }
Expand All @@ -46,13 +46,13 @@ public enum ServiceVersion
/// </summary>
/// <param name="version">
/// The <see cref="ServiceVersion"/> of the service API used when
/// making requests.
/// making requests.
/// </param>
public KeyClientOptions(ServiceVersion version = ServiceVersion.V7_0)
{
this.Version = version;
}

internal string GetVersionString()
{
var version = string.Empty;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -150,6 +150,5 @@ internal override void WriteProperties(Utf8JsonWriter json)
}

internal override void ReadProperties(JsonElement json) { }

}
}
Original file line number Diff line number Diff line change
Expand Up @@ -25,21 +25,20 @@ public class RsaKeyCreateOptions : KeyCreateOptions
/// </summary>
public int? KeySize { get; set; }

/// <summary>
/// <summary>
/// Determines whether or not a hardware key (HSM) is used for creation.
/// </summary>
///
/// <value><c>true</c> to use a hardware key; <c>false</c> to use a software key</value>
public bool Hsm { get; private set; }


/// <summary>
/// Initializes a new instance of the RsaKeyCreateOptions class.
/// </summary>
/// <param name="name">The name of the key.</param>
/// <param name="hsm">Whether to import as a hardware key (HSM) or software key.</param>
/// <param name="keySize">Key size in bits.</param>
public RsaKeyCreateOptions(string name, bool hsm, int? keySize = null)
public RsaKeyCreateOptions(string name, bool hsm, int? keySize = null)
{
Name = name;
Hsm = hsm;
Expand Down
3 changes: 1 addition & 2 deletions sdk/keyvault/Azure.Security.KeyVault.Secrets/src/Page.cs
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ internal override void ReadProperties(JsonElement json)
i++;
}
}

}

if (json.TryGetProperty("nextLink", out JsonElement nextLink))
Expand All @@ -68,7 +67,7 @@ internal override void ReadProperties(JsonElement json)

internal override void WriteProperties(ref Utf8JsonWriter json)
{
// serialization is not needed this type is only in responses
// serialization is not needed this type is only in responses
throw new NotImplementedException();
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ public SecretBase(string name)
/// Secret identifier.
/// </summary>
public Uri Id => _identifier.Id;

/// <summary>
/// Vault base URL.
/// </summary>
Expand Down Expand Up @@ -64,7 +64,7 @@ public SecretBase(string name)
public bool? Managed { get; private set; }

/// <summary>
/// If this is a secret backing a KV certificate, then this field specifies
/// If this is a secret backing a KV certificate, then this field specifies
/// the corresponding key backing the KV certificate.
/// </summary>
public string KeyId { get; private set; }
Expand Down