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

Add a field to the job summary to indicate if at least one bug was created. #3441

Closed
wants to merge 9 commits into from

Conversation

chkeita
Copy link
Contributor

@chkeita chkeita commented Aug 21, 2023

Add a field to the job summary to indicate if at least one bug was created.

closes #3548

@codecov-commenter
Copy link

codecov-commenter commented Aug 21, 2023

Codecov Report

Merging #3441 (3cc48d9) into main (d50fd48) will increase coverage by 7.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3441      +/-   ##
==========================================
+ Coverage   39.95%   47.14%   +7.18%     
==========================================
  Files         303      167     -136     
  Lines       37724    22918   -14806     
  Branches     1738        0    -1738     
==========================================
- Hits        15073    10804    -4269     
+ Misses      22330    12114   -10216     
+ Partials      321        0     -321     

see 138 files with indirect coverage changes

@chkeita chkeita force-pushed the store_notification branch 4 times, most recently from ab57da5 to c983ac6 Compare August 22, 2023 18:24
return the list of job created in the job_summary
@chkeita chkeita changed the title storing the ado notification title and work item id Storing the ado notification title and work item id Oct 10, 2023
@chkeita chkeita requested a review from tevoinea October 10, 2023 22:36
@chkeita chkeita marked this pull request as ready for review October 10, 2023 22:36

public IAsyncEnumerable<AdoNotificationEntry> GetByJobId(Guid jobId);

public Async.Task<bool> WasNotfied(Guid jobId);
Copy link
Member

Choose a reason for hiding this comment

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

NIT: typo WasNotified


public record AdoNotificationEntry(
[PartitionKey] Guid JobId,
[RowKey] int Id,
Copy link
Member

Choose a reason for hiding this comment

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

A work item ID needs an associated ADO org. We should store the ADO org here as well, otherwise it'll be very painful to figure out which org it was created in. Could we also name this WorkItemId so we know what the ID is for?

@chkeita chkeita changed the title Storing the ado notification title and work item id Add a field to the job summary to indicate if at least one bug was created. Oct 31, 2023
@chkeita chkeita closed this Oct 31, 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.

Include in the job summary whether or no a crash report was generated during the run
3 participants