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

linking author to profile #203

Merged
merged 2 commits into from
Mar 18, 2024
Merged

Conversation

chrheg
Copy link
Contributor

@chrheg chrheg commented Feb 5, 2024

Proposed Changes

  • Minor change, just linking the author to their GitHub Profile.

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run make lint and fix any issues that you have introduced
  • run make test and ensure you have test coverage for the lines you are introducing

Reviewer

  • Label as either bug, documentation, enhancement, infrastructure, or breaking

@chrheg chrheg requested a review from zkoppert as a code owner February 5, 2024 09:08
@zkoppert zkoppert added the enhancement New feature or request label Feb 6, 2024
@zkoppert
Copy link
Member

@chrheg Are you able to look at debugging the failing test?

@chrheg
Copy link
Contributor Author

chrheg commented Mar 17, 2024

@chrheg Are you able to look at debugging the failing test?

@zkoppert update the code now, make test passes locally now.

@zkoppert
Copy link
Member

Thanks @chrheg ! 🚀

@zkoppert zkoppert merged commit 6c119f8 into github:main Mar 18, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants