-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Merge main into live #10337
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
Merge main into live #10337
Conversation
Build.Reason:Manual by Min Huang Build.Url:https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=450660&view=results source_repo.branch:mh/workaround source_repo.url:https://apidrop.visualstudio.com/_git/binaries
Build.Reason:Manual by Min Huang Build.Url:https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=451135&view=results source_repo.branch:mh/workaround source_repo.url:https://apidrop.visualstudio.com/_git/binaries
Update to remove unexpected `)` characters that don't have a matching `(` character. Also, trim unnecessary nearby spaces.
Changed references to entryNameEncoding to clarify that it changes the encoding and decoding of both entry names and comments.
* Update sample.cs - Fix first if-statement in Create method: currently checks to see if `nameValue == null` **and** `nameValue.Length == 0`. Should check if one or the other. - Change exception thrown from above if-statement from `ArgumentNullException` to `ArgumentException` and replaced argument with `nameof` expression. - Remove unnecessary empty lines. * Apply suggestions from code review --------- Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
CI Update: Exclude CollectionsExtensions class from Microsoft.Extensions.DependencyModel.dll
Learn Build status updates of commit a61db45:
|

Please don't squash-merge this PR.