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

Resolving Regression Bugs - File Bugs on regression_report and properly reset state on duplicate. #3263

Merged
merged 11 commits into from
Jul 5, 2023

Conversation

nharper285
Copy link
Contributor

Summary of the Pull Request

What is this about?
We were not filing (or updating) bugs for regression reports.

Separately, there was a bug in our notification logic that prevented bugs from having their state reset.

PR Checklist

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

@codecov-commenter
Copy link

codecov-commenter commented Jun 29, 2023

Codecov Report

Merging #3263 (58b2cf3) into main (2bb627c) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #3263      +/-   ##
==========================================
- Coverage   29.65%   29.65%   -0.01%     
==========================================
  Files         332      332              
  Lines       39296    39302       +6     
==========================================
  Hits        11654    11654              
- Misses      27642    27648       +6     
Impacted Files Coverage Δ
...Service/ApiService/onefuzzlib/notifications/Ado.cs 9.85% <0.00%> (-0.17%) ⬇️

Copy link
Contributor

@stishkin stishkin left a comment

Choose a reason for hiding this comment

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

I'd wait for Teo's feedback though before merging

@Porges Porges requested a review from tevoinea July 3, 2023 21:18
@nharper285 nharper285 merged commit e2b34a0 into microsoft:main Jul 5, 2023
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request Jul 13, 2023
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.

5 participants