-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
some netherlands specific validations added #628
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
Conversation
Thanks. Can you look at the lint errors Travis lists? Run |
account = value.replace(/ /g,''); // remove spaces | ||
var sum = 0; | ||
var len = account.length; | ||
for (pos=0; pos<len; pos++) { |
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.
Should probably delete the line above and use this
for (var pos = 0, len = account.length; pos < len; pos++) {
You're right. I installed grunt, ran it and fixed the code. |
done, including the grunt install.... Grunt was right. robbert On Wed, Feb 6, 2013 at 3:18 PM, Jörn Zaefferer notifications@github.comwrote:
Robbert Wethmar |
This reverts commit a6ad15e.
this pull request is a bit of a mess now... I think i'll just until my other pull request (IBAN, pull request #630) is integrated, and then clean & redo this one. |
Thanks! Squashed into one commit, in 509e6e3 |
as discussed in email.