Heath Stewart

Results 419 comments of Heath Stewart

FWIW, we do have some basic guidance on sample authoring now at https://azure.github.io/azure-sdk/dotnet_introduction.html#dotnet-samples. I made some improvements - though there was significant feedback for additional work that is lower priority...

Alternatively, until we make a [comprehensive](https://github.com/Azure/azure-sdk-for-net/issues/33433) change, @ericstj recommended we just hardcode a fallback to the "netstandard2.0" assembly if "net461" doesn't exist. This should be a bit easier to handle...

The workaround has a problem where, becuase attributes were encoded slightly differently, you get errors like: > C:\Users\heaths\.nuget\packages\microsoft.dotnet.apicompat\5.0.0-beta.20467.1\build\Microsoft.DotNet.ApiCompat.targets(82,5): error : CannotChangeA ttribute : Attribute 'System.ComponentModel.EditorBrowsableAttribute' on 'Azure.Security.KeyVault.Administration.KeyVaultRoleScope.Equals(System.Ob ject)' changed from...

There are currently no plans to ship a net461 binary. We can revisit should that change.

Unfortunately, purging does take time and there's no way to poll for it like we do for deletions or recovery. This is a service limitation. @jlichwa for feedback. Is this...

Purging does work, though. You just can't expect to turn around and create a new key/secret/cert for ~30s in many cases. That seems like an impractical scenario. There is, unfortunately,...

@Zach-King2 go through the error and a couple warnings in https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2894774&view=logs&j=e37a1bb9-a190-5269-3a0e-37469dfbe682&t=09cb0672-20c5-5dce-4552-dad3f3e96fd3 and see if you can resolve those. For the missing file under eng/, just copy it. Assuming no changes...

> @heaths Could this be related to the compiler issue? Not sure. @allenjzhang can someone take a look? It seems files are up to date with what's in main. Long-term,...

@allenjzhang thank you. The PR is incomplete e.g., comments, because this is very much a work in progress; though, I agree something should be added as much as possible to...

Some failures are expected for now. @Zach-King2 will start a new PR to fill out most/all the remaining models that should fix almost all the remaining issues that aren't tooling-related.