Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Request for Approvers Role #97

Merged
merged 1 commit into from
May 5, 2022
Merged

Conversation

jacobweinstock
Copy link
Member

Hello. I would like to request the Approver role in the Hegel repo.

GitHub Username
@jacobweinstock

Role
Approver

Requirements:

  • approver from 2020-10-01 until 2022-04-01
  • reviewer from 2022-04-29 until present

Reviews:

Maintainers
@chrisdoherty4, @nshalman, @mmlb

Description

Why is this needed

Fixes: #

How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

Hello. I would like to request the Approver role in the Hegel repo.

GitHub Username
@jacobweinstock

Role
Approver

Requirements:
- approver from 2020-10-01 until 2022-04-01 
- reviewer from 2022-04-29 until present

Reviews:
- tinkerbell#87
- tinkerbell#66
- tinkerbell#65
- tinkerbell#60

Maintainers
@chrisdoherty4, @nshalman, @mmlb
@codecov
Copy link

codecov bot commented May 5, 2022

Codecov Report

Merging #97 (79c38d0) into main (beab761) will not change coverage.
The diff coverage is n/a.

❗ Current head 79c38d0 differs from pull request most recent head 3be7198. Consider uploading reports for the commit 3be7198 to get more accurate results

@@           Coverage Diff           @@
##             main      #97   +/-   ##
=======================================
  Coverage   51.87%   51.87%           
=======================================
  Files           3        3           
  Lines         401      401           
=======================================
  Hits          208      208           
  Misses        164      164           
  Partials       29       29           

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 beab761...3be7198. Read the comment docs.

Copy link
Member

@chrisdoherty4 chrisdoherty4 left a comment

Choose a reason for hiding this comment

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

Great to have your input and experience, thanks!

@nshalman nshalman added the ready-to-merge Signal to Mergify to merge the PR. label May 5, 2022
@mergify mergify bot merged commit a509fa4 into tinkerbell:main May 5, 2022
@jacobweinstock jacobweinstock deleted the patch-1 branch June 24, 2022 03:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants