-
Notifications
You must be signed in to change notification settings - Fork 357
Merge release 1.16 into master #1584
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
Open
WhitWaldo
wants to merge
23
commits into
master
Choose a base branch
from
release-1.16
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Updated Directory.Build.props to drop .NET 6 and .NET 7 targets * Removed explicit .NET targets in favor of using Directory.Build.props * Added Directory.Build.props for all examples, updated examples to favor targeting this file instead * Updated README to include missing packages and reflect a few modernization updates (including the new target versions) * Fixed file numbering in doc * Fixed comment on global.json to reflect use of the 9.0.100 minimum SDK version * Removed .net 6 and 7 targets from GitHub build script * Updated integration test build script to remove .NET 7 and .NET 6 targets Signed-off-by: Whit Waldo <[email protected]>
* Updated all documentation referring to .NET 6 and .NET 7 to refer instead to .NET 8 and higher. * Updated minimum langversion to reflect C# 12 (released with .NET 8) Signed-off-by: Whit Waldo <[email protected]>
* Implemented generic Dapr client builder method to use across all Dapr packages to reduce code duplication throughout. * Implemented changes to Dapr.AI to use generic client builder extension. * Updating IDaprClient to implement IDisposable instead of putting it on each client type * Modified Conversation client to utilize the GrpcClient pattern of the other clients for consistency. * Updated Dapr.Jobs to use generic builder extensions * Refactored Dapr.Messaging for consistency with other Dapr clients (e.g. implements IDisposable, primary constructors, use of generic service registration extension) * Tweak to use same import reference as other clients for consistency * Renamed to AddDaprConversationClient to remain consistent with 1.14 naming - no need to introduce a breaking change here. Signed-off-by: Whit Waldo <[email protected]>
* Removed unnecessary comment * Update to use file-level namespaces * Using target-typed new expression * QoL improvements to code base - primary constructors on classes and records, use of collection initializers, target-typed new expressions, etc. * Applied all manner of QoL improvements to Dapr.Client * Fixed break in example caused by nullability annotation change * Refactored where streaming subscription example is located in solution to align with packages * Refactored actor example to modernize * Modernized example Signed-off-by: Whit Waldo <[email protected]>
…tributes necessary for System.Text.Json support (#1498) Signed-off-by: Whit Waldo <[email protected]>
* Removed obsolete project from solution (Messaging) * Added project containing analyzers and tests with respect to Dapr.Actors. The starting analyzer in this package validates that there's a method on the actor type matching the name provided in the timer callback argument. Signed-off-by: Whit Waldo <[email protected]>
Added Roslyn analyzers and code fixes for: - Encouraging JSON serialization for actors - Validating actors are properly registered in DI - Ensuring that the actor handler mapping is performed during startup Signed-off-by: Nils Gruson <[email protected]> Signed-off-by: Whit Waldo <[email protected]> Co-authored-by: Whit Waldo <[email protected]>
… jobs are scheduled (#1477) * Add Analyzer * Refactored for consistent naming. Updated to use Resources instead of constant strings and to share values via internal values where possible. Added location to job names that trigger diagnostic and updated verbiage of formatted value. Co-authored-by: Siri Varma Vegiraju <[email protected]> Co-authored-by: Whit Waldo <[email protected]>
* Removed erroneous duplicate project directory * Corrected project name for consistency across test projects
* Added initial pass at documentation for the various analyzers. Placing the documentation link above the "Error handling" section in leftnav. * Updated diagnostic IDs throughout solution to match values in documentation Signed-off-by: Whit Waldo <[email protected]>
Introduced two diagnostic analyzers: `WorkflowRegistrationAnalyzer` and `WorkflowActivityRegistrationAnalyzer` that validate that both any types that implement `Workflow<,>` or `WorkflowActivity<,>` have a corresponding registration entry in the DI provider. This PR also includes a code fix provider for either analyzer that provides a remedial registration for the developer. Signed-off-by: Nils Gruson <[email protected]> Co-authored-by: Whit Waldo <[email protected]>
…ts (#1437) * Instead of just attaching metadata to the full envelope, the metadata is attached to each individual message as well, if it exists at all. Signed-off-by: Chris Walsh <[email protected]> Co-authored-by: Whit Waldo <[email protected]>
#1524) * Updated solution to remove unnecessary dependencies on Newtonsoft.Json in favor of just using System.Text.Json throughout for JSON serialization.
* Implementation of the new crypto client Signed-off-by: Whit Waldo <[email protected]> Co-authored-by: Christopher Watford <[email protected]>
Signed-off-by: Whit Waldo <[email protected]>
Updated Jobs SDK to reflect both the overwrite and failure policy additions
* Replaced Obsolete with Experimental attributes throughout solution, added documentation and updated unit tests
Merging all the hotfix patches from master into release-1.16 to prepare for release
* Today, a publish can happen without actually waiting on integration tests to pass. This changes this behavior. * Added validation requirement to Publish Packages to require that workflow_run dependency Signed-off-by: Whit Waldo <[email protected]>
* Added example to demonstrate parallel workflow concurrency
Signed-off-by: Whit Waldo <[email protected]>
* Fixed build errors
* Added example demonstrating use of Dapr with Aspire across two projects (using service invocation) Signed-off-by: Whit Waldo <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
In preparation to release the SDK v1.16, this merges everything from this release back into master
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: