Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Handle another duplicate field #3383

Merged

Conversation

tevoinea
Copy link
Member

@tevoinea tevoinea commented Aug 3, 2023

Summary of the Pull Request

What is this about?

Closes #3382

Adds duplicate checking for "Microsoft.VSTS.Common.ResolvedReason"

@codecov-commenter
Copy link

codecov-commenter commented Aug 3, 2023

Codecov Report

Merging #3383 (52e83e0) into main (dde7a09) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #3383      +/-   ##
==========================================
- Coverage   32.21%   32.21%   -0.01%     
==========================================
  Files         302      302              
  Lines       36927    36928       +1     
==========================================
  Hits        11897    11897              
- Misses      25030    25031       +1     
Files Changed Coverage Δ
...Service/ApiService/onefuzzlib/notifications/Ado.cs 27.09% <0.00%> (-0.07%) ⬇️

@tevoinea tevoinea enabled auto-merge (squash) August 3, 2023 18:24
@tevoinea tevoinea merged commit ab189b1 into microsoft:main Aug 3, 2023
AdamL-Microsoft pushed a commit that referenced this pull request Aug 3, 2023
* Handle another duplicate field

* Make case insensitive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Handle when duplicate reason is stored in Microsoft.VSTS.Common.ResolvedReason field
4 participants