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

Handle another duplicate field#3383

Merged
tevoinea merged 2 commits into
microsoft:mainfrom
tevoinea:tevoinea/HandleAnotherDuplicateField
Aug 3, 2023
Merged

Handle another duplicate field#3383
tevoinea merged 2 commits into
microsoft:mainfrom
tevoinea:tevoinea/HandleAnotherDuplicateField

Conversation

@tevoinea

@tevoinea tevoinea commented Aug 3, 2023

Copy link
Copy Markdown
Contributor

Summary of the Pull Request

What is this about?

Closes #3382

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

Comment thread src/ApiService/ApiService/onefuzzlib/notifications/Ado.cs Outdated
@codecov-commenter

codecov-commenter commented Aug 3, 2023

Copy link
Copy Markdown

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