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

Can create ado notifications #2456

Merged
merged 3 commits into from
Sep 28, 2022

Conversation

tevoinea
Copy link
Member

Summary of the Pull Request

What is this about?

Ports #2435 to C# and makes ADO notifications work. Makes progress towards #1754

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

  1. Submit a new job
  2. Create a new ADO template based on the reports container for the job you just created
  3. Check that work items are being created

@codecov-commenter
Copy link

codecov-commenter commented Sep 28, 2022

Codecov Report

Merging #2456 (2ae31b1) into main (872c107) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main    #2456   +/-   ##
=======================================
  Coverage   29.88%   29.88%           
=======================================
  Files         288      288           
  Lines       35183    35180    -3     
=======================================
  Hits        10515    10515           
+ Misses      24668    24665    -3     
Impacted Files Coverage Δ
...ce/ApiService/onefuzzlib/NotificationOperations.cs 0.00% <0.00%> (ø)
...Service/ApiService/onefuzzlib/notifications/Ado.cs 0.00% <0.00%> (ø)
...piService/onefuzzlib/notifications/GithubIssues.cs 0.00% <0.00%> (ø)
...vice/onefuzzlib/notifications/NotificationsBase.cs 0.00% <0.00%> (ø)
...rvice/ApiService/onefuzzlib/notifications/Teams.cs 0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@tevoinea tevoinea merged commit 9a04272 into microsoft:main Sep 28, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Oct 28, 2022
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.

3 participants