Radi Totev
Radi Totev
@ieahleen, @Sboonny I removed the checks for `let` and `var`. Here are my reasons: * it appears there is no satisfactory way of removing JS comments. * we don't want...
@Sembauke I can see a few more files from the description haven't been migrated. Are you sure you want to close this issue?
@ieahleen this doesn't work, [see here](https://github.com/freeCodeCamp/freeCodeCamp/pull/46753#pullrequestreview-1028861387): >I have tried to cover for this test case with reassigning the variables and expecting an error but this didn't work as well: >assert.throws(()...
Thanks for opening this issue @Sboonny .
I have tried this as well but it doesn't work. In the test reassigning the value of any of the constants doesn't throw, e.g.: ```js try { LOGIN = "LOGOUT"...
There are two conditions to be fulfilled for this test: 1. User should use variable instead of string 2. Variable should be constant Consider this: ```js assert(LOGIN === 'LOGIN' &&...
@ieahleen I was waiting to hear any comments regarding the proposed changes but I assume whoever saw this is ok. We can open this for contributors. The change should be...
@SelloFotoyi I would suggest updating the [README](https://github.com/Yog9/SnapShot/blob/master/README.md) file to be included in this PR. Lines 22-23.
> Thanks @raditotev. I'm on it. By the way, do you have any open source ReactJS projects (for beginners) you could refer me to? I can't think of any. If...
Why is this PR still open?