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

Add repo roles: #94

Merged
merged 1 commit into from
Apr 29, 2022
Merged

Add repo roles: #94

merged 1 commit into from
Apr 29, 2022

Conversation

jacobweinstock
Copy link
Member

@jacobweinstock jacobweinstock commented Apr 29, 2022

Description

This is needed to enable the repo roles defined by the community here: tinkerbell/org#10

CC @chrisdoherty4

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

This is needed to enable the repo roles defined by
the community here: tinkerbell/org#10

Signed-off-by: Jacob Weinstock <[email protected]>
@codecov
Copy link

codecov bot commented Apr 29, 2022

Codecov Report

Merging #94 (c935dcd) into main (c7de39b) will not change coverage.
The diff coverage is n/a.

❗ Current head c935dcd differs from pull request most recent head 362a1b9. Consider uploading reports for the commit 362a1b9 to get more accurate results

@@           Coverage Diff           @@
##             main      #94   +/-   ##
=======================================
  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 c7de39b...362a1b9. Read the comment docs.

@mmlb mmlb added the ready-to-merge Signal to Mergify to merge the PR. label Apr 29, 2022
@jacobweinstock jacobweinstock added ready-to-merge Signal to Mergify to merge the PR. and removed ready-to-merge Signal to Mergify to merge the PR. labels Apr 29, 2022
@mergify mergify bot merged commit beab761 into tinkerbell:main Apr 29, 2022
@jacobweinstock jacobweinstock deleted the role-bootstrap branch April 29, 2022 15:41
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.

3 participants