Missing semi-colons on the below lines of file public/js/calculator.js:
https://github.com/freeCodeCamp/freeCodeCamp/blob/staging/public/js/calculator.js#L186
https://github.com/freeCodeCamp/freeCodeCamp/blob/staging/public/js/calculator.js#L203
https://github.com/freeCodeCamp/freeCodeCamp/blob/staging/public/js/calculator.js#L246
Note: Beginner friendly only.
just curious, is this causing a bug? I would think ASI would handle this, but it would make for a good PR for beginners.
Well, it's truly not required, but yes for the sake of keeping things consistent why not. Thanks, @aashil, and @jonathanihm!
@jonathanihm I don't think it's a bug in any way. Think of it as a task for first timers. I will be submitting more like these to help people make their first contribution.
Consistency and best standards are best!
Oh first timer's this issue is about knowing how to setup and make a pull request. Here are the
Guidelines for Contributing.
I'll take it... @raisedadead 馃槇
Please let us know if you need any help! Cheers!
@CEREBR4L Awesome!!!
Can i make the change and do a pull request?