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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve Author Detection #3119

Merged
merged 2 commits into from
Oct 5, 2022
Merged

Improve Author Detection #3119

merged 2 commits into from
Oct 5, 2022

Conversation

AyanSinhaMahapatra
Copy link
Member

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled 馃搼 and links the original issue above 馃敆
  • Tests pass -- look for a green checkbox 鉁旓笍 a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 馃搧

pombredanne and others added 2 commits October 4, 2022 18:51
@pombredanne
Improve author detection
4b41354 
This is form of author with just a first name and email.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@AyanSinhaMahapatra
Fix test failures with ignorable authors
ac06b60 
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra AyanSinhaMahapatra changed the base branch from develop to prep-release-31-2 October 4, 2022 19:15
@pombredanne pombredanne merged commit ac06b60 into prep-release-31-2 Oct 5, 2022
@pombredanne pombredanne deleted the author-name-email branch October 24, 2022 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@AyanSinhaMahapatra @pombredanne