Library Review for Kado 4.0
Code Review Guide Lines
The libraries need a complete code review from the following aspects:
- Completeness
- Matching tests
- Matching documentation (see doc/api/.md)
- Correct ES6 code
- No ES5 constructs (no usage of var etc)
- No obvious security issues
- No global scope usage
- Syntax style adherence (including a right margin of 80 lines)
- Correct license header
- License adherence (including contributing code)
Library Review Checklist
-
Application -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Asset -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Child -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Cluster -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Command -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
CommandServer -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Connector -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Cron -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Database -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Email -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
EmailEngine -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Event -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Format -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
GetOpt -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
History -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
HyperText -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
HyperTextEngine -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Language -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Library -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Logger -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Mapper -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Message -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Module -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Navigation -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Permission -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Profiler -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Router -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Search -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
TestRunner -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Util -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
Validate -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-
-
View -
Completeness -
Matching tests -
Matching documentation (see doc/api/.md) -
Correct ES6 code -
No ES5 constructs (no usage of var etc) -
No obvious security issues -
No global scope usage -
Syntax style adherence (including a right margin of 80 lines) -
Correct license header -
License adherence (including contributing code)
-