stephentoub
Follow
Pinned
2,624 contributions in the last year
Activity overview
Contributed to
dotnet/runtime,
microsoft/semantic-kernel,
dotnet/roslyn-analyzers
and 26 other
repositories
Contribution activity
May 2023
Created 20 commits in 3 repositories
Created a pull request in dotnet/runtime that received 12 comments
Add Base64.IsValid and allow Base64.DecodeXx methods to skip whitespace
Cherrypicks, addresses merge conflicts, and squashes the commits from @heathbm in #79334, then adds an additional commit to clean up a few things i…
+1,080
−45
•
12
comments
Opened 16 other pull requests in 2 repositories
dotnet/runtime
3
open
6
merged
- Remove generated locals in RegexCompiler.LoadSearchValues
- Add ArgumentException.ThrowIfNullOrWhiteSpace
- Dedup PemEncoding.TryCountBase64 with Base64.IsValid
- Remove Vector<T> fallbacks for Vector128<T>
- Use string.IsNullOrEmpty and ArgumentException.ThrowIfNullOrEmpty in many more places
- Avoid using two-char search in IndexOfOrdinalIgnoreCase for 1-char values
- Annotate IHtmlString.ToHtmlString's return as nullable
- Simplify SelectArrayIterator method
- Add back IHtmlString to System.Web.HttpUtility
microsoft/semantic-kernel
7
merged
- Make all protected members of ClientBase / OpenAIClientBase / AzureOpenAIClientBase private protected
- Fix a few NotNull calls
- Fix a few analyzer diagnostics
- Remove unnecessary suppressions and fix a few warnings
- Some clean up in OpenAIClientBase
- Remove duplicative extension methods on KernelPlanExtensions
- Remove ValidationException
Reviewed 35 pull requests in 6 repositories
dotnet/runtime
20 pull requests
- Remove generated locals in RegexCompiler.LoadSearchValues
- Fix null handling of built-in converters
- Add ArgumentException.ThrowIfNullOrWhiteSpace
- Fix ReadOnlySpanGetReferenceAndReadInteger on BigEndian
- Add ToDictionary() overloads for KeyValuePair and ValueTuple kv.
- Use string.IsNullOrEmpty and ArgumentException.ThrowIfNullOrEmpty in many more places
- Add Base64.IsValid and allow Base64.DecodeXx methods to skip whitespace
-
Remove last
!from Caching.Abstractions - Remove check from FillStringChecked and rename it to CopyStringContent
- Fix JsonNode.To allocations
- Use Vector128 and Vector256 in interface cast
- Bugfix/30024 problem with concurrency type descriptor getconverter
- [wasm] Update path in the WBT README
- Add back IHtmlString to System.Web.HttpUtility
- Fix TimeZoneInfo Perf
- Rename IndexOfAnyValues to SearchValues
- JIT: Intrinsify UTF16->UTF8 conversion for string literals (Encoding.UTF8.TryGetBytes)
- Fix new warnings found with CA1854 improvement.
- Fix typo in SafeHandle.cs
- Improve vectorization of IndexOf(chars, StringComparison.OrdinalIgnoreCase)
microsoft/semantic-kernel
11 pull requests
- Allow to replace functions (fix regression)
- Simplify orchestration API
- Simplify IKernel interface
- Fix a few analyzer diagnostics
- Some clean up in OpenAIClientBase
- Fix code style
- Remove unnecessary suppressions and fix a few warnings
- SKFunction Invoke Parameter Duplication
- Remove ValidationException
- Azure Cognitive Search connector for Semantic Memory
- Fix potential multiple enumerations
dotnet/performance
1 pull request
DotNetAnalyzers/StyleCopAnalyzers
1 pull request
dotnet/dotnet-api-docs
1 pull request
dotnet/docs
1 pull request
Created an issue in dotnet/runtime that received 5 comments
[API Proposal]: Remove struct constraint from Vector*<T>
The constraint makes it much more challenging to use. For example, in a generic method that supports working with arbitrary Ts and in which you wan…
5
comments