-
Notifications
You must be signed in to change notification settings - Fork 11
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: harvest PR attribution from squash commits #349
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good!
# A tibble: 2 x 4 | ||
description type breaking scope | ||
<chr> <chr> <lgl> <lgl> | ||
1 blop (@person) Uncategorized FALSE NA |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At some point we can store this in a structured way. Not a show-stopper here.
author_lines <- description_lines[grepl(author_pattern(), description_lines)] | ||
authors <- rematch2::re_match(author_lines, "<.*@users.noreply.github.com>")$.match | ||
authors <- stats::na.omit(authors) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One re_match()
call might be able to handle all of that, again, not a showstopper.
Fix #344
Very related to #343 hence the priority bump 馃槄
TODOs