-
Notifications
You must be signed in to change notification settings - Fork 1
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
Improve scanning #14
Improve scanning #14
Conversation
camalot
commented
Feb 2, 2017
- updated the regex patterns
- added git commit history scanning
- broke up portions in to re-usable functions
- added tests for the new functions
- improved some existing tests
- added git commit history scanning - broke up portions in to re-usable functions - added tests for the new functions - improved some existing tests
Hi! I'm your friendly Peer Review Bot. For this Pull Request to be labeled as "peer-reviewed", you'll need at least 1 comments containing the magic phrase "LGTM" or ":+1:". If someone replies with "Needs Work" or ":-1:", that same user will need to reply again with indicating they approve of the changes. |
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.
LGTM