-
Notifications
You must be signed in to change notification settings - Fork 781
Add commit author #547
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 commit author #547
Conversation
@clayreimann Do you want me to merge from upstream master into my branch prior to merging? |
@j-rewerts merge or rebase is fine with me. My first instinct is to squash merge for small PRs with more than 1 commit. |
I borrowed this strategy from the writeFile approach.
Codecov Report
@@ Coverage Diff @@
## master #547 +/- ##
=========================================
Coverage ? 90.79%
=========================================
Files ? 12
Lines ? 413
Branches ? 0
=========================================
Hits ? 375
Misses ? 38
Partials ? 0
Continue to review full report at Codecov.
|
d25bc31
to
09f2042
Compare
@clayreimann Yeah feel free to squash and merge. |
Closes #546.
The commit function can now accept optional parameters for setting who the author or committer is. See here for GitHub's API on it.