Skip to content

Conversation

@Zoooook
Copy link

@Zoooook Zoooook commented Nov 21, 2014

Cleaned up a few things, also fixed an error in the AZ description.

@Zoooook Zoooook changed the title Cleaned up a few things Fully updated and cleaned up, also in Groovy now, with tests. Nov 25, 2014
@adambullmer
Copy link
Owner

Thank you so much for your hard work on this! Weird github didn't email me about an open PR.

A couple of pieces of feedback before I'd feel comfortable merging this:

  • Fix merge conflicts
  • Squash commits down into 1 commit for this PR
  • It looks like some of your rules deleted some validation rules. Can you please make sure all of your rules match all of the rules from https://ntsi.com/drivers-license-format/

@Zoooook
Copy link
Author

Zoooook commented Apr 20, 2016

Looks like the merge conflicts are due to your recent overhaul of the system. I last touched this a year and a half ago for a project at work, and I'm not terribly interested in it anymore. All I did was clean up the regex in some cases and change the rules in other cases to comply with the 2014 MVR Access and Decoder Digest, as noted in my commit messages.

I would assume that our 2 sources for validation rules are in conflict in some cases. You're welcome to verify those rules by looking through my unit tests. I would however question the authoritativeness of NTSI.

It may be better to just leave my branch as a fork; my main interest was getting a groovy version that complied with the source we had to use at work. Updating the regex in your format was just a bonus. Feel free to steal my regex, or not, as you see fit.

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.

2 participants