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.

Sign up for GitHub

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

Jump to bottom

Updating with new employer #276

Merged
merged 2 commits into from
Aug 15, 2023
Merged

Updating with new employer #276

merged 2 commits into from
Aug 15, 2023

Conversation

TheFoxAtWork
Copy link
Contributor

Updating my affiliation to reflect my new employer.

@TheFoxAtWork
Updating with new employer
be07466 
Updating my affiliation to reflect my new employer.
@krook krook added the ok-to-test Apply if PR is validated safe to test label Aug 15, 2023
@krook
Copy link
Member

krook commented Aug 15, 2023

@TheFoxAtWork Congrats!

Two notes before merging though:

  • Can you please sign the commit so the DCO bot is happy?
  • Can you please add a space to make it "Red Hat"?

@TheFoxAtWork
Copy link
Contributor Author

Yep working on it now, thought i had it all set up but apparently not.

@TheFoxAtWork
correcting spelling
2bae7dc 
Signed-off-by: Emily Fox <themoxiefoxatwork@gmail.com>
@TheFoxAtWork
Copy link
Contributor Author

@krook all done!

@krook krook merged commit 4257588 into cncf:main Aug 15, 2023
2 checks passed
@krook
Copy link
Member

krook commented Aug 15, 2023

Thank you @TheFoxAtWork!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@krook krook krook approved these changes

Assignees
No one assigned
Labels
ok-to-test Apply if PR is validated safe to test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TheFoxAtWork @krook