Skip to content

Increase DTFx.Core and DTFx.AS versions#1053

Merged
davidmrdavid merged 2 commits intomainfrom
dajusto/rev-vers
Mar 18, 2024
Merged

Increase DTFx.Core and DTFx.AS versions#1053
davidmrdavid merged 2 commits intomainfrom
dajusto/rev-vers

Conversation

@davidmrdavid
Copy link
Collaborator

As in title, routine PR for upcoming release

@davidmrdavid davidmrdavid requested a review from bachuv March 18, 2024 22:23
<MajorVersion>1</MajorVersion>
<MinorVersion>17</MinorVersion>
<PatchVersion>1</PatchVersion>
<PatchVersion>2</PatchVersion>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I recommend we exclude this change from the release since 1) it appears we didn't make any changes to DurableTask.AzureStorage and 2) the change to DurableTask.Core doesn't apply to DurableTask.AzureStorage (AFAICT, the entity-specific error handling only applies to backends like MSSQL that don't yet implement out-of-proc entity support).

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah I felt similarly that this was a rather light release so it makes sense not to release DTFx.AS at all. Incorporated your feedback.

Copy link
Member

@cgillum cgillum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@davidmrdavid davidmrdavid merged commit 027e8ae into main Mar 18, 2024
@davidmrdavid davidmrdavid deleted the dajusto/rev-vers branch March 18, 2024 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants