-
Notifications
You must be signed in to change notification settings - Fork 290
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
Coding problem #185
Comments
Hey @rechmbrs, thanks for your input. If you think that line is erroneous, please provide a test or two as proof. |
Alexandru,
The line would not pass a Lint test. I haven't gotten the file to work yet
and went to Lint to make sure I have a chance of it working. I'll try some
more but there is still a problem overall. I don't know if the suggestion
I make is correct but it passes the Lint test.
RONC
…On Mon, Jan 6, 2020 at 4:26 AM Alexandru Mărășteanu < ***@***.***> wrote:
Hey @rechmbrs <https://github.com/rechmbrs>, thanks for your input. If
you think that line is erroneous, please provide a test or two as proof.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#185?email_source=notifications&email_token=ABAYIOUO3KDEVLFGEGXY353Q4MBORA5CNFSM4KCWX2WKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIFBOBI#issuecomment-571086597>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABAYIOTGL3IITBATOAGARDDQ4MBORANCNFSM4KCWX2WA>
.
--
*Ronald E Chambers1713 Twila LnConroe, TX 77301-1175C - 713.806.6304E
- rechmbrs@gmail.com <rechmbrs@gmail.com>*
|
You can open a PR, though the current linter doesn't mind that line |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I believe that line: pad_character = ph.pad_char ? ph.pad_char === '0' ? '0' : ph.pad_char.charAt(1) : ' '
should be: pad_character = ph.pad_char ? (ph.pad_char === '0' ? '0' : ph.pad_char.charAt(1)) : ' '
Missing parenthesis.
RONC
The text was updated successfully, but these errors were encountered: