Skip to content

Conversation

williambdean
Copy link
Collaborator

@williambdean williambdean commented Dec 25, 2024

Describe what this PR does / why we need it

Display the author of the commit if the commit user is NIL

Does this pull request fix one issue?

Closes #731
Related to #707

Describe how you did it

  • Pull down the data
  • Add logic to the writer

Describe how to verify it

Take a PR that has merge commit from the GitHub UI

For example: #702

Special notes for reviews

Checklist

  • Passing tests and linting standards
  • Documentation updates in README.md and doc/octo.txt

@williambdean williambdean linked an issue Dec 25, 2024 that may be closed by this pull request
Copy link

@chisung9999 chisung9999 left a comment

Choose a reason for hiding this comment

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

병합요청

@williambdean williambdean merged commit aa3cb8e into master Dec 27, 2024
4 checks passed
@williambdean williambdean deleted the 731-merge-commit-author-is-not-shown branch December 27, 2024 17:04
@williambdean williambdean mentioned this pull request Dec 28, 2024
2 tasks
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.

Merge commit author is not shown
2 participants