-
Notifications
You must be signed in to change notification settings - Fork 52
Open
Labels
Description
Per MDN Code Style Guideines will need to update/check the following:
- Arrays are aligned
- Array item additions use
.push() - Comments match
//and/* */guidelines - Comments after
console.log()andconsole.error()usage- Update
console.log()to useconsole.error()when appropriate
- Update
- Function alignment
- camelCase
- declaration
- Update anonymous / arrow functions
- For loops
-
.forEach()usage - Update
for(;;)versions to be only used for updating; however,- Note semantic iteration options that can be used instead where possible
-
- Add in
switchstatements - Update
try/catchblocks - Object/class updates not needed in ICS3U as this is an ICS4U concept
- Even though guidelines suggestion using the ternary
?operator, will not use in ICS3U for readability - Update notes on strict equality
===/!==versus loose equality==/!= - Update
console.log()and strings to use templates{a}style for majority of work - Variable names should be okay, but review to be sure
- Update type coersion
+val, and"" + numberto useNumber()andString()instead - API work unnecessary in ICS3U notes
Reactions are currently unavailable