Skip to content

MOB-19642#149

Merged
sbenedicadb merged 2 commits intoadobe:dev-v4.0.2from
sbenedicadb:MOB-19642
Oct 11, 2023
Merged

MOB-19642#149
sbenedicadb merged 2 commits intoadobe:dev-v4.0.2from
sbenedicadb:MOB-19642

Conversation

@sbenedicadb
Copy link
Copy Markdown
Member

@sbenedicadb sbenedicadb commented Oct 10, 2023

Description

  • fixes a potential threading issue when using the raw request APIs
  • adds thread safe access to TargetState containers

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@sbenedicadb sbenedicadb requested a review from swarna04 October 10, 2023 20:03
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 10, 2023

Codecov Report

Merging #149 (2777175) into dev-v4.0.2 (2628a73) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@              Coverage Diff               @@
##           dev-v4.0.2     #149      +/-   ##
==============================================
+ Coverage       92.13%   92.17%   +0.04%     
==============================================
  Files              25       25              
  Lines            2186     2196      +10     
==============================================
+ Hits             2014     2024      +10     
  Misses            172      172

pfransenadb
pfransenadb approved these changes Oct 10, 2023
View reviewed changes
Copy link
Copy Markdown

@pfransenadb pfransenadb left a comment

Choose a reason for hiding this comment

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

👍

@sbenedicadb sbenedicadb merged commit 2347309 into adobe:dev-v4.0.2 Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pfransenadb pfransenadb pfransenadb approved these changes

@swarna04 swarna04 Awaiting requested review from swarna04

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sbenedicadb @pfransenadb