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

Convert ADO traces to customEvents and update notificationInfo #2508

Merged

Conversation

nharper285
Copy link
Contributor

Summary of the Pull Request

What is this about?
Adding project and name to ADO notifications.

PR Checklist

  • Applies to work item: Add Project and Name Job Details to Notifications Events. #2507
  • CLA signed. If not, go over here and sign the CLI.
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

@codecov-commenter
Copy link

codecov-commenter commented Oct 12, 2022

Codecov Report

Merging #2508 (513ff4a) into main (51b8dc0) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #2508      +/-   ##
==========================================
- Coverage   29.96%   29.96%   -0.01%     
==========================================
  Files         289      289              
  Lines       35421    35424       +3     
==========================================
  Hits        10615    10615              
- Misses      24806    24809       +3     
Impacted Files Coverage Δ
...Service/ApiService/onefuzzlib/notifications/Ado.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.

@nharper285 nharper285 changed the title Adding Project and Name to ADO Notification. Convert ADO traces to events and update notificationInfo Oct 12, 2022
@nharper285 nharper285 changed the title Convert ADO traces to events and update notificationInfo Convert ADO traces to customEvents and update notificationInfo Oct 12, 2022
@nharper285 nharper285 requested a review from stishkin October 13, 2022 00:39
@nharper285 nharper285 marked this pull request as ready for review October 13, 2022 00:39
chkeita and others added 3 commits October 13, 2022 16:13
* Update the logic update the scaleinProtection

* build fix
* Add SrcView::try_insert function

* Make try_insert return bool indicating success

* Rustfmt

* Set test to be ignored (no compile)

* Bump srcview version

Co-authored-by: Sutton Bradley <suttonb@microsoft.com>
Co-authored-by: George Pollard <gpollard@microsoft.com>
@nharper285 nharper285 force-pushed the user/noharper/notification-job-details branch from 4502f38 to e49a9ef Compare October 13, 2022 16:14
@nharper285 nharper285 merged commit 3c35b34 into microsoft:main Oct 13, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Nov 13, 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.

7 participants