Skip to content

Add additional reviewers to min and latest dependency checker#1558

Merged
gsheni merged 6 commits intomainfrom
update_reviewers_deps_checker
Jul 26, 2021
Merged

Add additional reviewers to min and latest dependency checker#1558
gsheni merged 6 commits intomainfrom
update_reviewers_deps_checker

Conversation

@gsheni
Copy link
Contributor

@gsheni gsheni commented Jul 26, 2021

  • Add additional people to review automated MRs relating to changes to dependencies.

@gsheni gsheni requested a review from jeff-hernandez July 26, 2021 15:03
@gsheni gsheni self-assigned this Jul 26, 2021
@codecov
Copy link

codecov bot commented Jul 26, 2021

Codecov Report

Merging #1558 (8fdde01) into main (5bf3f05) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1558   +/-   ##
=======================================
  Coverage   98.58%   98.58%           
=======================================
  Files         140      140           
  Lines       15003    15003           
=======================================
  Hits        14791    14791           
  Misses        212      212           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5bf3f05...8fdde01. Read the comment docs.

@rwedge
Copy link
Contributor

rwedge commented Jul 26, 2021

should we try using the team-reviewers option instead and maintain the list of reviewers that way?

Copy link
Contributor

@jeff-hernandez jeff-hernandez left a comment

Choose a reason for hiding this comment

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

Looks good!

@gsheni gsheni merged commit e9d036b into main Jul 26, 2021
@rwedge rwedge deleted the update_reviewers_deps_checker branch July 27, 2021 19:25
@simha104 simha104 mentioned this pull request Aug 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants