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.

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

Many changes to reduce false positives and more: #11

Merged
merged 1 commit into from
Nov 29, 2017

Conversation

justbbetter
Copy link
Contributor

  • made DIGITS more restrictive: only 4+ consecutive digits
  • match NAME only in saluation and valediction contexts
  • company names are not really sensitive to don't redact them
  • add SSN pattern

@coveralls
Copy link

coveralls commented Nov 28, 2017

Coverage Status

Coverage remained the same at 96.97% when pulling 16afec3 on justin-reduce-false-positives into 33d9ce1 on master.

* made DIGITS more restrictive: only 4+ consecutive digits
* match NAME only in saluation and valediction contexts
* company names are not really sensitive to don't redact them
* add SSN pattern
@coveralls
Copy link

coveralls commented Nov 29, 2017

Coverage Status

Coverage remained the same at 96.97% when pulling bfc41a7 on justin-reduce-false-positives into 33d9ce1 on master.

@justbbetter justbbetter merged commit 769a414 into master Nov 29, 2017
@justbbetter justbbetter deleted the justin-reduce-false-positives branch November 29, 2017 00:37
@patrickhulce
Copy link
Contributor

just a friendly tip for next time since it seems like a few others have picked up the library (#10), when options are removed it's nice to publish in new major version (just need to put BREAKING CHANGE: blah blah in the commit description for it to pick it up automatically)

solvvy ❤️s OSS 😄 🎉

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.

4 participants