Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

"Check for palindromes" passes with full uppercasing as well

See original GitHub issue

Challenge Check for Palindromes has an issue. User Agent is: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.84 Safari/537.36. Please describe how to reproduce this issue, and include links to screenshots if possible.

My code:

function palindrome(str) {
  str = str.toUpperCase().replace(/\W|_/g, '');
  return str === str.split('').reverse().join('');
}
palindrome("eye");

A minor issue, but since the instructions state it should be lowercased, a test for lowercasing should be added. Currently it works as long as the case is the same throughout.

Issue Analytics

State:
Created 7 years ago
Comments:6 (5 by maintainers)

Top GitHub Comments

1reaction

weejimmycommented, Aug 5, 2016

I can do this one.

0reactions

BKinahancommented, Aug 5, 2016

Great! 👍

To potential contributors: This issue can be resolved by opening a pull request that changes line 171 of this file as detailed above. Please follow the Contributor’s Guide and remember to visit the HelpContributors chat room if you need any assistance. We’re here to help!